Review Request 108213: [2/3] Adjust KDevelop to EnvironmentSelectionWidget refactor.

[Ivan Shapovalov]

> On July 1, 2013, 5:43 p.m., Milian Wolff wrote:
> > projectmanagers/custom-buildsystem/custombuildsystemconfigwidget.cpp, line 94
> > <http://git.reviewboard.kde.org/r/108213/diff/2/?file=166108#file166108line94>
> >
> >     Here and above/below: QString() instead of ""?

Just for the sake of uniformity and not touching unrelated things. May do in a separate commit if you wish :)


- Ivan


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108213/#review35385
-----------------------------------------------------------


On June 27, 2013, 12:58 p.m., Ivan Shapovalov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108213/
> -----------------------------------------------------------
> 
> (Updated June 27, 2013, 12:58 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Description
> -------
> 
> A follow-up to https://git.reviewboard.kde.org/r/108212/.
> This removes calls to addItems() and uses "" instead of "default" as default config values.
> 
> 
> Diffs
> -----
> 
>   projectmanagers/custom-buildsystem/custombuildsystemconfigwidget.cpp 6934c43 
>   projectmanagers/custom-buildsystem/configwidget.cpp bcd85c5 
>   projectmanagers/cmake/settings/cmakepreferences.cpp c309fef 
>   projectbuilders/makebuilder/makejob.cpp 1a8c2ce 
>   projectbuilders/makebuilder/makeconfig.ui 592bba3 
>   projectbuilders/makebuilder/makebuilderpreferences.cpp 5ffceb6 
> 
> Diff: http://git.reviewboard.kde.org/r/108213/diff/
> 
> 
> Testing
> -------
> 
> Manual testing.
> 
> 
> Thanks,
> 
> Ivan Shapovalov
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20130702/69609d78/attachment.html>