Description
For a long time, there has been the so-called "Protected Hack". On many platforms, it is possible to #define protected public and to simplify the generated code this way.
Unfortunately, this trick does not work under Windows, and we have to do quite some type casts to get the desired effect, when things are protected.
But there is no known problem that cannot be solved without the "Protected Hack", in so far I don't see any need to keep this hack. Additionally, this hack generates a code split between Windows and Unix-alikes. It happened recently that a patch went through and broke Windows, because we forgot about this hack.
As a first step, I will disable the hack, i.E. remove the --avoid-protected-hack flag and define m_avoidProtectedHack = true;. Later, after activation of the CI system, the variable will be completely evicted.
Attachments
For Gerrit Dashboard: PYSIDE-504 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
192418,2 | Remove the "protected hack”, part 1(2) | 5.6 | pyside/pyside-setup | Status: ABANDONED | +1 | 0 |
192419,4 | Disable the "protected hack” | 5.6 | pyside/pyside-setup | Status: ABANDONED | 0 | 0 |
193106,8 | Fix headergenerator to build under C++11 | 5.6 | pyside/pyside-setup | Status: MERGED | +2 | 0 |
193107,8 | Remove fixup_headers after fixing shiboken for C++11 | 5.6 | pyside/pyside-setup | Status: MERGED | +2 | 0 |
332842,3 | shiboken6: Enable building with --avoid-protected-hack for gcc, clang | dev | pyside/pyside-setup | Status: MERGED | +2 | 0 |