> On July 12, 2014, 6:49 p.m., David Faure wrote: > > Seems clear enough. > > > > In KF5 the file is in frameworks/kwindowsystem/src/kwindowinfo_mac.cpp > > Marko Käning wrote: > Thanks for the info about KF5! > > Which branch should this be committed to on kdelibs4? Simply on master or > also on (an)other branch(es)?
Commit to KDE/4.13, and merge (or ask someone to merge) to KDE/4.14 and master. [Or if you don't care for the 4.13 branch on mac, commit to KDE/4.14 directly and merge to master.] - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119240/#review62201 ----------------------------------------------------------- On July 12, 2014, 6:17 p.m., Marko Käning wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119240/ > ----------------------------------------------------------- > > (Updated July 12, 2014, 6:17 p.m.) > > > Review request for kdelibs, David Faure, Martin Gräßlin, Ian Wadham, and > Thomas Lübking. > > > Bugs: 337154 > http://bugs.kde.org/show_bug.cgi?id=337154 > > > Repository: kdelibs > > > Description > ------- > > Preventing a crash in the KWindowInfo::Private destructor on OSX (thanks to > René J.V. Bertin) for Qt4! > > (Unclear is atm where this would have to be fixed in KF5.) > > See also this: > https://developer.apple.com/library/mac/documentation/CoreFoundation/Reference/CFTypeRef/Reference/reference.html#//apple_ref/c/func/CFRelease > > > Diffs > ----- > > kdeui/windowmanagement/kwindowinfo_mac.cpp > ca502a3c643af36f4a52bb6dcbc7b54fbe3f42a9 > > Diff: https://git.reviewboard.kde.org/r/119240/diff/ > > > Testing > ------- > > For more details see associated bug > > > Thanks, > > Marko Käning > >