D15532: kdev-astyle : improved ObjC support

René J.V. Bertin noreply at phabricator.kde.org
Thu Sep 20 10:52:35 BST 2018

rjvbb marked 11 inline comments as done.
rjvbb added inline comments.


> pino wrote in astyle_plugin.cpp:36
> `static const char foo[] = "..."`, which is more efficient (makes the whole string const)

And there I was thinking the days were gone where foo[] and *foo were ever so subtly different... :)

> kossebau wrote in astyle_plugin.cpp:284
> What about "ObjC" -> "Objective-C"? That is the id also used for X-KDevelop-Languages, would be good to be consistent.
> Also is this the string currently used for display, so not using an abbreviation might be better.

Thanks, that's a nice reminder there a few more places where changes are required and for which I have D15530 <https://phabricator.kde.org/D15530> up!

> kossebau wrote in astyle_plugin.cpp:327
> While adding this line, already add a space between if and (.

I take it you meant fixing the whitespace for the entire method(?)

> kfunk wrote in astyle_plugin.cpp:377
> No `break;` needed after return.

I know, I always add them anyway in my own code (just like I always add a default: with a break). Saves headscratching later on when you refactor and miss a missing break...

  R32 KDevelop


To: rjvbb, #kdevelop, kossebau, kfunk
Cc: kfunk, pino, kossebau, kdevelop-devel, glebaccon, antismap, iodelay, vbspam, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20180920/5c15e8e2/attachment.html>

More information about the KDevelop-devel mailing list