<div dir="ltr"><p dir="ltr">I agree with Daniel,</p><p>Also I suggest we keep this change for right after the 1.5 release, usually there's less work going on after the release. so Andrea do it then, please.</p><p><br></p><p>Thanks,</p><p>Mehrdad</p><p><br></p>
<div class="gmail_quote">On Dec 13, 2014 12:39 AM, "Daniel Kreuter" <<a href="mailto:daniel.kreuter85@gmail.com" target="_blank">daniel.kreuter85@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>Hi Andrea,<br><br></div>I'd suggest first review the pending requests/patches and then reformat the code.<br><br></div>Every new request has to be reworked to fit the KDE coding style.<br><br>Regards<br><br>Daniel<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Dec 12, 2014 at 5:07 PM, Andrea Scarpino <span dir="ltr"><<a href="mailto:scarpino@kde.org" target="_blank">scarpino@kde.org</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi there,<br>
I guess do you agree with me when I say that code formatting in Choqok sucks.<br>
2 spaces, 4 spaces, tabs, if (), if() if( foo), if (foo)... I guess we have<br>
every coding style in this single project.<br>
<br>
I propose to run some formatting tool which follow KDE coding style (which<br>
should be the same of Qt). Unfortunately those stuff break pending<br>
reviews/patches.<br>
<br>
What do you think about this situation?<br>
<br>
Cheers<br>
<span><font color="#888888"><br>
--<br>
Andrea<br>
_______________________________________________<br>
Choqok-devel mailing list<br>
<a href="mailto:Choqok-devel@kde.org" target="_blank">Choqok-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/choqok-devel" target="_blank">https://mail.kde.org/mailman/listinfo/choqok-devel</a><br>
</font></span></blockquote></div></div>
<br>_______________________________________________<br>
Choqok-devel mailing list<br>
<a href="mailto:Choqok-devel@kde.org" target="_blank">Choqok-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/choqok-devel" target="_blank">https://mail.kde.org/mailman/listinfo/choqok-devel</a><br>
<br></blockquote></div>
</div>