> On Aug. 19, 2013, 5:06 p.m., David Faure wrote: > > This patch looks OK. > > > > Deprecating it, I'm not sure. Please investigate porting some kde4 apps > > from createGUI to setupGUI to see what's actually involved in that.
This is what is involved. Less code, less bugs. http://quickgit.kde.org/?p=krdc.git&a=commit&h=0233f23b99095d641336fb6fad080b62852ddcc9 - Albert ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112144/#review38159 ----------------------------------------------------------- On Aug. 18, 2013, 2:15 p.m., Albert Astals Cid wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/112144/ > ----------------------------------------------------------- > > (Updated Aug. 18, 2013, 2:15 p.m.) > > > Review request for kdelibs and David Faure. > > > Description > ------- > > Suggest people to use setupGUI instead of createGUI. > > Also mention in setHelpMenuEnabled that it will affect setupGUI > > I was going to suggest to actually deprecate createGUI but maybe it is too > much for 4.x, but maybe in frameworks? > > > Diffs > ----- > > kdeui/xmlgui/kxmlguiwindow.h 3237b8c > > Diff: http://git.reviewboard.kde.org/r/112144/diff/ > > > Testing > ------- > > N/A > > > Thanks, > > Albert Astals Cid > >
