Hi Boudewijn,

these kind of refactorings are neccessary to keep code maintainable I fear.

Anyway, it seems like you are maintaining a bunch of custom patches on top of Qt? How about reducing your maintainance burden by upstreaming them? Qt would benefit from bugfixes and new features. You would benefit from reduced workload and responsiblity? This sounds like win-win to me, do you agree?

Ciao,
Mathias

Am 12.10.2018 um 12:48 schrieb b...@valdyas.org:
So, there's this commit:

https://git.qt.io/consulting-usa/qtbase-xcb-rendering/commit/69335920f724d2d4a49924f373c4fef57c942831

"
Move QButtonGroup implementation from qabstractbutton.cpp to qbuttongroup.cpp

Because it's the right thing to do.

Needed to introduce qbuttongroup_p.h because QAbstractButton
likes to poke around in QButtonGroup's private parts.

Fixed includes of qabstractbutton_p.h so it compiles on it's
own.

Change-Id: Ic7725277d2419754de273b2abd4790476edd0eb4
Reviewed-by: Olivier Goffart's avatarOlivier Goffart (Woboq GmbH) <ogoff...@woboq.com>

"

Which of course breaks source compatibility. It's bad enough to have to adapt one's codebase; but this also makes it impossible to bisect code when Qt 5.11 is installed that had to be adapted.

I'm constantly running into this problem. Would it be too much to ask that things like this just never ever happen again?


Boudewijn
_______________________________________________
Development mailing list
Development@qt-project.org
http://lists.qt-project.org/mailman/listinfo/development
_______________________________________________
Development mailing list
Development@qt-project.org
http://lists.qt-project.org/mailman/listinfo/development

Reply via email to