D17729: KateStatusBar: Reformatted by astyle command to follow coding style

loh tar noreply at phabricator.kde.org
Thu Dec 27 08:03:26 GMT 2018


loh.tar added a comment.


  @cullmann wrote:
  
  > Actually, if there is a astyle commandline that enforces the right style, could you paste it here?
  
  Um (?) I had in mind it was on the Kate HP but couln'd find it now. But it's there https://community.kde.org/Policies/Kdelibs_Coding_Style#Artistic_Style_.28astyle.29_automatic_code_formatting
  
  As said I add "--align-pointer=name" but don't like to change this wiki, as noob :-)
  Furthermore I noticed that it is still not full coding style conform due to not fixed "one line conditional statements", so "--add-braces" may an option but I didn't try it
  
  > I am in favor of some script that runs that over the repo and just applying that regularly to enforce it stays sane, that is much more useful than to review each ( placement in all commits ;=)
  
  In generally a good idea. But sometimes it's nice to break the coding style to achieve some more lovely look

REPOSITORY
  R39 KTextEditor

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

To: loh.tar, #ktexteditor, dhaumann, cullmann
Cc: cullmann, sars, kwrite-devel, kde-frameworks-devel, #ktexteditor, hase, michaelh, ngraham, bruns, demsking, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20181227/b680a232/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list