Review Request 128159: Suggest to use setupGUI() instead of setXMLFile()

[Friedrich W. H. Kossebau]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128159/#review102950
-----------------------------------------------------------



Not using "Create" with the setupGUI() call though ran across the initial idea of this patch,
as now setXMLFile() is no longer called implicitly when the "Create" flag is not passed to
setupGUI() :)

Filed https://phabricator.kde.org/D5181 to fix this up.

- Friedrich W. H. Kossebau


On June 13, 2016, 5:17 p.m., Elvis Angelaccio wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128159/
> -----------------------------------------------------------
> 
> (Updated June 13, 2016, 5:17 p.m.)
> 
> 
> Review request for KDE Frameworks and David Faure.
> 
> 
> Repository: kparts
> 
> 
> Description
> -------
> 
> KXMLGui-based apps should always use `setupGUI()`, which has several advantages over `setXMLFile()` and calls it implicitly. 
> 
> This patch also adds a code snippet about this.
> 
> 
> Diffs
> -----
> 
>   src/mainwindow.h b672a35cba92475200ff15620a21150da19582dd 
> 
> Diff: https://git.reviewboard.kde.org/r/128159/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Elvis Angelaccio
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170326/13d2354f/attachment.html>