|
|
(9 intermediate revisions by 3 users not shown) |
Line 1: |
Line 1: |
| This page is meant as examples of things you cannot do in C++ when maintaining [[Policies/Binary Compatibility Issues With C++|binary compatibility]].
| | {{Moved To Community}} |
| | |
| == Unexport or remove a class ==
| |
| {| | |
| |-
| |
| ! Before
| |
| ! After
| |
| |-
| |
| | <code cppqt>
| |
| class KDECORE_EXPORT KUrl
| |
| { | |
| // [...]
| |
| };
| |
| </code>
| |
| | <code cppqt>
| |
| class KUrl
| |
| {
| |
| // [...]
| |
| };
| |
| </code>
| |
| |}
| |
| | |
| '''Reason''': the symbols for the class above are not added to the exported symbols list of the library, so other libraries and applications cannot see them.
| |
| | |
| == Change the class hierarchy ==
| |
| {|
| |
| |-
| |
| ! Before
| |
| ! After
| |
| |-
| |
| | <code cppqt>
| |
| class MyClass: public BaseClass
| |
| {
| |
| // [...]
| |
| };
| |
| </code>
| |
| | <code cppqt>
| |
| class MyClass: public BaseClass, public OtherBaseClass
| |
| {
| |
| // [...]
| |
| };
| |
| </code>
| |
| |-
| |
| | <code cppqt>
| |
| class MyClass: public BaseClass1, public BaseClass2
| |
| {
| |
| // [...]
| |
| };
| |
| </code>
| |
| | <code cppqt>
| |
| class MyClass: public BaseClass2, public BaseClass1
| |
| {
| |
| // [...]
| |
| };
| |
| </code>
| |
| |}
| |
| | |
| '''Reason''': the size and/or order of member data in the class changes, causing existing code to allocate too much or too little memory, read/write data at the wrong offsets.
| |
| | |
| == Change the template arguments of a template class ==
| |
| {|
| |
| |-
| |
| ! Before
| |
| ! After
| |
| |-
| |
| | <code cppqt>
| |
| template<typename T1>
| |
| class MyTemplateClass
| |
| {
| |
| // [...]
| |
| };
| |
| </code>
| |
| | <code cppqt>
| |
| template<typename T1, typename T2 = void>
| |
| class MyTemplateClass
| |
| {
| |
| // [...]
| |
| };
| |
| </code>
| |
| |}
| |
| | |
| <code cppqt>
| |
| // GCC mangling before: _Z3foo15MyTemplateClassIiE
| |
| // after: _Z3foo15MyTemplateClassIivE
| |
| void foo(MyTemplateClass<int>);
| |
| </code>
| |
| | |
| '''Reason''': the mangling of the functions related to this template type change because its template expansion changes too. This can happen both for member functions (for example, the constructor) as well as functions that take it as a parameter.
| |
| | |
| == Unexport a function ==
| |
| {|
| |
| |-
| |
| ! Before
| |
| ! After
| |
| |-
| |
| | <code cppqt>
| |
| Q_CORE_EXPORT const char *qVersion();
| |
| </code>
| |
| | <code cppqt>
| |
| const char *qVersion();
| |
| </code>
| |
| |-
| |
| | <code cppqt>
| |
| namespace KSocketFactory {
| |
| KDECORE_EXPORT QTcpSocket *connectToHost(...);
| |
| }
| |
| </code>
| |
| | <code cppqt>
| |
| namespace KSocketFactory {
| |
| QTcpSocket *connectToHost(...);
| |
| }
| |
| </code>
| |
| |}
| |
| | |
| '''Reason''': the symbols for the functions above are not added to the exported symbols list of the library, so other libraries and applications cannot see them.
| |
| | |
| == Inline a function ==
| |
| {|
| |
| |-
| |
| ! Before
| |
| ! After
| |
| |-
| |
| | <code cppqt>
| |
| int square(int n);
| |
| </code>
| |
| | <code cppqt>
| |
| inline int square(int n) { return n * n; }
| |
| </code>
| |
| |-
| |
| | <code cppqt>
| |
| int square(int n) { return n * n; }
| |
| </code>
| |
| | <code cppqt>
| |
| inline int square(int n) { return n * n; }
| |
| </code>
| |
| |-
| |
| | <code cppqt>
| |
| class Math
| |
| {
| |
| int square(int n);
| |
| };
| |
| | |
| // the following could be in a .cpp
| |
| int Math::square(int n)
| |
| { return n * n; }
| |
| </code>
| |
| | <code cppqt>
| |
| class Math
| |
| {
| |
| int square(int n);
| |
| };
| |
| | |
| // the following could be in a .cpp
| |
| inline int Math::square(int n)
| |
| { return n * n; }
| |
| </code>
| |
| |-
| |
| | <code cppqt>
| |
| class Math
| |
| {
| |
| int square(int n);
| |
| };
| |
| | |
| // the following could be in a .cpp
| |
| int Math::square(int n)
| |
| { return n * n; }
| |
| </code>
| |
| | <code cppqt>
| |
| class Math
| |
| {
| |
| int square(int n)
| |
| { return n * n; }
| |
| };
| |
| </code>
| |
| |}
| |
| | |
| '''Reason''': when a function is declared inline and the compiler does inline it at its call point, the compiler does not have to emit an out-of-line copy. Code that exists and was calling this function will therefore not be able to resolve the function anymore. Also, when compiling with GCC and <tt>-fvisibility-inlines-hidden</tt>, if the compiler does emit an out-of-line copy, it will be hidden (not added to the exported symbols table) and therefore not accessible from other libraries.
| |
| | |
| == Change the parameters of a function ==
| |
| {|
| |
| |-
| |
| ! Before
| |
| ! After
| |
| |-
| |
| | <code cppqt>
| |
| // GCC mangling: _Z11doSomethingii
| |
| // MSVC mangling: ?doSomething@@YAXHH@Z
| |
| void doSomething(int i1, int i2);
| |
| </code>
| |
| | <code cppqt>
| |
| // GCC mangling: _Z11doSomethingis
| |
| // MSVC mangling: ?doSomething@@YAXHF@Z
| |
| void doSomething(int i1, short i2);
| |
| </code>
| |
| |-
| |
| | <code cppqt>
| |
| // GCC mangling: _Z11doSomethingii
| |
| // MSVC mangling: ?doSomething@@YAXHH@Z
| |
| void doSomething(int i1, int i2);
| |
| </code>
| |
| | <code cppqt>
| |
| // GCC mangling: _Z11doSomethingiii
| |
| // MSVC mangling: ?doSomething@@YAXHHH@Z
| |
| void doSomething(int i1, int i2, int i3 = 0);
| |
| </code>
| |
| |-
| |
| | <code cppqt>
| |
| // GCC mangling: _Z11doSomethingRi
| |
| // MSVC mangling: ?doSomething@@YAXABH@Z
| |
| void doSomething(int &i1);
| |
| </code>
| |
| | <code cppqt>
| |
| // GCC mangling: _Z11doSomethingRKi
| |
| // MSVC mangling: ?doSomething@@YAXAAH@Z
| |
| void doSomething(const int &i1);
| |
| </code>
| |
| |-
| |
| | <code cppqt>
| |
| void doSomething(int i1);
| |
| </code>
| |
| | <code cppqt>
| |
| void doSomething(const int i1); // breaks with Sun CC
| |
| </code>
| |
| |-
| |
| | <code cppqt>
| |
| // GCC mangling: _Z11doSomethingPc
| |
| // MSVC mangling: ?doSomething@@YAXPAD@Z (32-bit)
| |
| void doSomething(char *ptr);
| |
| </code>
| |
| | <code cppqt>
| |
| // GCC mangling: _Z11doSomethingPKc
| |
| // MSVC mangling: ?doSomething@@YAXPBD@Z (32-bit)
| |
| void doSomething(const char *ptr);
| |
| </code>
| |
| |}
| |
| | |
| == Change the return type ==
| |
| {|
| |
| |-
| |
| ! Before
| |
| ! After
| |
| |-
| |
| | <code cppqt>
| |
| // GCC mangling: _Z8positionv
| |
| // MSVC mangling: ?position@@YA_JXZ
| |
| qint64 position();
| |
| </code>
| |
| | <code cppqt>
| |
| // GCC mangling: _Z8positionv
| |
| // MSVC mangling: ?position@@YAHXZ
| |
| int position();
| |
| </code>
| |
| |-
| |
| | <code cppqt>
| |
| // GCC mangling: _Z4namev
| |
| // MSVC mangling: ?position@@YAVQByteArray@@DXZ
| |
| QByteArray name();
| |
| </code>
| |
| | <code cppqt>
| |
| // GCC mangling: _Z4namev
| |
| // MSVC mangling: ?position@@YAVQString@@XZ
| |
| QString name();
| |
| </code>
| |
| |-
| |
| | <code cppqt>
| |
| // GCC mangling: _Z4namev
| |
| // MSVC mangling: ?position@@YAPBDXZ
| |
| const char *name();
| |
| </code>
| |
| | <code cppqt>
| |
| // GCC mangling: _Z4namev
| |
| // MSVC mangling: ?position@@YAVQString@@XZ
| |
| QString name();
| |
| </code>
| |
| |-
| |
| | <code cppqt>
| |
| // GCC mangling: _Z12createDevicev
| |
| // MSVC mangling: ?createDevice@@YAPAVQTcpSocket@@XZ
| |
| QTcpSocket *createDevice();
| |
| </code>
| |
| | <code cppqt>
| |
| // GCC mangling: _Z12createDevicev (unchanged)
| |
| // MSVC mangling: ?createDevice@@YAPAVQIODevice@@XZ
| |
| QIODevice *createDevice();
| |
| </code>
| |
| |-
| |
| | <code cppqt>
| |
| // GCC mangling: _ZNK10QByteArray2atEi
| |
| // MSVC mangling: ?at@QByteArray@@QBA?BDH@Z
| |
| const char QByteArray::at(int) const;
| |
| </code>
| |
| | <code cppqt>
| |
| // GCC mangling: _ZNK10QByteArray2atEi (unchanged)
| |
| // MSVC mangling: ?at@QByteArray@@QBADH@Z
| |
| char QByteArray::at(int) const;
| |
| </code>
| |
| |}
| |
| | |
| '''Reason''': changing the return type changes the mangled name of the function names in some compilers (GCC notably does not encode the return type). However, even if the mangling doesn't change, the convention on how the return types are handled may change. In the first example above, the return type changed from a 64- to a 32-bit integer, which means on some architectures, the upper half of the return register may contain garbage. In the second example, the return type changed from QByteArray to QString, which are two incompatible types, whereas in the third example, the return type changed from a simple integer (a POD) to a QString -- in this case, the compiler usually needs to pass a hidden implicit first parameter, which won't be there.
| |
| | |
| == Change the access rights ==
| |
| {|
| |
| |-
| |
| ! Before
| |
| ! After
| |
| |-
| |
| | <code cppqt>
| |
| class MyClass
| |
| {
| |
| protected:
| |
| // GCC mangling: _ZN7MyClass11doSomethingEv
| |
| // MSVC mangling: ?doSomething@MyClass@@IAAXXZ
| |
| void doSomething();
| |
| };
| |
| </code>
| |
| | <code cppqt>
| |
| class MyClass
| |
| {
| |
| public:
| |
| // GCC mangling: _ZN7MyClass11doSomethingEv (unchanged)
| |
| // MSVC mangling: ?doSomething@MyClass@@QAAXXZ
| |
| void doSomething();
| |
| };
| |
| </code>
| |
| |}
| |
| | |
| '''Reason''': some compilers encode the protection type of a function in its mangled name.
| |
| | |
| == Change the CV-qualifiers of a member function ==
| |
| {|
| |
| |-
| |
| ! Before
| |
| ! After
| |
| |-
| |
| | <code cppqt>
| |
| class MyClass
| |
| {
| |
| public:
| |
| // GCC mangling: _ZNK7MyClass9somethingEv
| |
| // MSVC mangling: ?something@MyClass@QBAHXZ
| |
| int something() const;
| |
| };
| |
| </code>
| |
| | <code cppqt>
| |
| class MyClass
| |
| {
| |
| public:
| |
| // GCC mangling: _ZN7MyClass9somethingEv
| |
| // MSVC mangling: ?something@MyClass@QAAHXZ
| |
| int something();
| |
| };
| |
| </code>
| |
| |}
| |
| | |
| '''Reason''': compilers encode the constness of a function in the mangled name. The reason they all do that is because the C++ standard allows overloading of functions that differ only by the constness.
| |
| | |
| == Change the type of global data ==
| |
| {|
| |
| |-
| |
| ! Before
| |
| ! After
| |
| |-
| |
| | <code cppqt>
| |
| // GCC mangling: data (undecorated)
| |
| // MSVC mangling: ?data@@3HA
| |
| int data = 42;
| |
| </code>
| |
| | <code cppqt>
| |
| // GCC mangling: data (undecorated)
| |
| // MSVC mangling: ?data@@3FA
| |
| short data = 42;
| |
| </code>
| |
| |-
| |
| | <code cppqt>
| |
| class MyClass
| |
| {
| |
| public:
| |
| // GCC mangling: _ZN7MyClass4dataE
| |
| // MSVC mangling: ?data@MyClass@@2HA
| |
| static int data;
| |
| };
| |
| </code>
| |
| | <code cppqt>
| |
| class MyClass
| |
| {
| |
| public:
| |
| // GCC mangling: _ZN7MyClass4dataE
| |
| // MSVC mangling: ?data@MyClass@@2FA
| |
| static short data;
| |
| };
| |
| </code>
| |
| |}
| |
| | |
| '''Reason''': some compilers encode the type of the global data in its mangled name. Especially note that some compilers mangle even for simple data types that would be allowed in C, meaning the <tt>extern "C"</tt> qualifier makes a difference too.
| |
| | |
| == Change the CV-qualifiers of global data ==
| |
| {|
| |
| |-
| |
| ! Before
| |
| ! After
| |
| |-
| |
| | <code cppqt>
| |
| // MSVC mangling: ?data@@3HA
| |
| int data = 42;
| |
| </code>
| |
| | <code cppqt>
| |
| // MSVC mangling: ?data@@3HB
| |
| const int data = 42;
| |
| </code>
| |
| |-
| |
| | <code cppqt>
| |
| class MyClass
| |
| {
| |
| public:
| |
| // MSVC mangling: ?data@MyClass@@2HA
| |
| static int data;
| |
| };
| |
| </code>
| |
| | <code cppqt>
| |
| class MyClass
| |
| {
| |
| public:
| |
| // MSVC mangling: ?data@MyClass@@2HB
| |
| static const int data;
| |
| };
| |
| </code>
| |
| |-
| |
| | <code cppqt>
| |
| class MyClass
| |
| {
| |
| public:
| |
| static int data;
| |
| };
| |
| </code>
| |
| | <code cppqt>
| |
| class MyClass
| |
| {
| |
| public:
| |
| // the compiler won't even create a symbol
| |
| static const int data = 42;
| |
| };
| |
| </code>
| |
| |}
| |
| | |
| '''Reason''': some compilers encode the CV-qualifiers of the global data in its mangled name. Especially note that a static const value declared in the class itself can be considered for "inlining" -- that is, the compiler doesn't need to generate an external symbol for the value since all implementations are guaranteed to know it.
| |
| | |
| == Add a virtual member function to a class without any ==
| |
| {|
| |
| |-
| |
| ! Before
| |
| ! After
| |
| |-
| |
| | <code cppqt>
| |
| struct Data
| |
| {
| |
| int i;
| |
| };
| |
| </code>
| |
| | <code cppqt>
| |
| struct Data
| |
| {
| |
| int i;
| |
| virtual int j();
| |
| };
| |
| </code>
| |
| |}
| |
| | |
| '''Reason''': a class without any virtual members or bases is guaranteed to be exactly like a C structure, for compatibility with that language (that is a POD structure). On some compilers, structures/classes with bases that are POD themselves are also POD. However, as soon as there's one virtual base or virtual member function, the compiler is free to arrange the structure in a C++ manner, which usually means inserting a hidden pointer at the beginning or the end of the structure, pointing to the virtual table of that class. This changes the size and offset of the elements in the structure.
| |
| | |
| == Add new virtuals to a non-leaf class ==
| |
| {|
| |
| |-
| |
| ! Before
| |
| ! After
| |
| |-
| |
| | <code cppqt>
| |
| class MyClass
| |
| {
| |
| public:
| |
| virtual ~MyClass();
| |
| virtual void foo();
| |
| };
| |
| </code>
| |
| | <code cppqt>
| |
| class MyClass
| |
| {
| |
| public:
| |
| virtual ~MyClass();
| |
| virtual void foo();
| |
| virtual void bar();
| |
| };
| |
| </code>
| |
| |}
| |
| | |
| '''Reason''': the addition of a new virtual function to a class that is non-leaf (that is, there is at least one class deriving from this class) changes the layout of the virtual table (the virtual table is basically a list of function pointers, pointing to the functions that are active in this class level). To accommodate the new virtual, the compiler must add a new entry to this table, but existing derived classes won't know about it and will not have the entry in their virtual tables.
| |
| | |
| == Change the order of the declaration of virtual functions ==
| |
| {|
| |
| |-
| |
| ! Before
| |
| ! After
| |
| |-
| |
| | <code cppqt>
| |
| class MyClass
| |
| {
| |
| public:
| |
| virtual ~MyClass();
| |
| virtual void foo();
| |
| virtual void bar();
| |
| };
| |
| </code>
| |
| | <code cppqt>
| |
| class MyClass
| |
| {
| |
| public:
| |
| virtual ~MyClass();
| |
| virtual void bar();
| |
| virtual void foo();
| |
| };
| |
| </code>
| |
| |}
| |
| | |
| '''Reason''': the compiler places the pointers to the functions implementing the virtual functions in the order that they are declared in the class. By changing the order of the declaration, the order of the entries in the virtual table changes too.
| |
| | |
| Note: the order is inherited from the parent classes, so overriding a virtual will allocate the entry in the parent's order.
| |
| | |
| == Override a virtual that doesn't come from a primary base ==
| |
| <code cppqt>
| |
| class PrimaryBase
| |
| {
| |
| public:
| |
| virtual ~PrimaryBase();
| |
| virtual void foo();
| |
| };
| |
| | |
| class SecondaryBase
| |
| {
| |
| public:
| |
| virtual ~PrimaryBase();
| |
| virtual void bar();
| |
| };
| |
| </code>
| |
| {|
| |
| |-
| |
| ! Before
| |
| ! After
| |
| |-
| |
| | <code cppqt>
| |
| class MyClass: public PrimaryBase, public SecondaryBase
| |
| {
| |
| public:
| |
| ~MyClass();
| |
| void foo();
| |
| };
| |
| </code>
| |
| | <code cppqt>
| |
| class MyClass: public PrimaryBase, public SecondaryBase
| |
| {
| |
| public:
| |
| ~MyClass();
| |
| void foo();
| |
| void bar();
| |
| };
| |
| </code>
| |
| |}
| |
| | |
| '''Reason''': this is a tricky case. When dealing with multiple-inheritance of classes with virtual tables, the compiler must create multiple virtual tables to guarantee polymorphism works (that is, when your MyClass object is stored in a PrimaryBase or SecondaryBase pointer). The virtual table for the primary base is shared with the class's own virtual table, because they have the same layout at the beginning. However, if you override a virtual coming from a non-primary base, it is the same as adding a new virtual, since that primary base did not have the virtual by that name.
| |
| | |
| == Override a virtual with a covariant return with different top address ==
| |
| <code cppqt>
| |
| struct Data1 { int i; };
| |
| class BaseClass
| |
| {
| |
| public:
| |
| virtual Data1 *get();
| |
| };
| |
| | |
| struct Data0 { int i; };
| |
| struct Complex1: Data0, Data1 { };
| |
| struct Complex2: virtual Data1 { };
| |
| </code>
| |
| {|
| |
| |-
| |
| ! Before
| |
| ! After
| |
| |-
| |
| | <code cppqt>
| |
| class MyClass: public BaseClass
| |
| {
| |
| public:
| |
| };
| |
| </code>
| |
| | <code cppqt>
| |
| class MyClass: public BaseClass
| |
| {
| |
| public:
| |
| Complex1 *get();
| |
| };
| |
| </code>
| |
| |-
| |
| | <code cppqt>
| |
| class MyClass: public BaseClass
| |
| {
| |
| public:
| |
| };
| |
| </code>
| |
| | <code cppqt>
| |
| class MyClass: public BaseClass
| |
| {
| |
| public:
| |
| Complex2 *get();
| |
| }; | |
| </code>
| |
| |}
| |
| | |
| '''Reason''': this is another tricky case, like the above one and also for the same reason: the compiler must add a second entry to the virtual table, just as if a new virtual function had been added, which changes the layout of the virtual table and breaks derived classes.
| |
| | |
| Covariant calls happen when the function overriding a virtual from a parent class returns a class different from the parent (this is allowed by the C++ standard, so the code above is perfectly valid and calling <tt>p->get()</tt> with p of type <tt>BaseClass</tt> will call <tt>MyClass::get</tt>). If the more-derived type doesn't have the same top-address such as <tt>Complex1</tt> and <tt>Complex2</tt> above, when compared to <tt>Data1</tt>, then the compiler needs to generate a stub function (usually called a "thunk") to adjust the value of the pointer returned. It places the address to that thunk in the entry corresponding to the parent's virtual function in the virtual table. However, it also adds a new entry for calls made which return the new top-address.
| |