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