D17278: Meson plugin: First working version

Daniel Mensinger noreply at phabricator.kde.org
Sat Dec 1 18:10:00 GMT 2018


dmensinger marked 4 inline comments as done.
dmensinger added inline comments.

INLINE COMMENTS

> rizzitello wrote in mesonconfigpage.cpp:178
> In these cases the type is obvious. Why not use auto?

Then I guess it is personal preference :)
I like it better this way for (relatively) short type names without templates and `::` (outside C++11 for loops).
If you are OK with it, I would like to keep the types the way they are now.

REPOSITORY
  R32 KDevelop

REVISION DETAIL
  https://phabricator.kde.org/D17278

To: dmensinger, apol
Cc: rizzitello, yurchor, kde-doc-english, kdevelop-devel, glebaccon, hase, antismap, iodelay, vbspam, skadinna, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20181201/4e0cf837/attachment.html>


More information about the KDevelop-devel mailing list