On Thu, Mar 5, 2015 at 12:56 AM, Jaroslaw Staniek <stan...@kde.org> wrote:
> Nice thanks.
> I see 28% of changes are non-whitespace.
> Forcing {} for if/else is good.
>
> Could we please exclude dirs that contain forked code that would be
> compared to external copies?
> One comes to my mind: kexi/migration/mdb/src/mdbtools/. Anything else?
>
> For the future I'd use the tool as a local pre-commit hook.
>
> For stats I recommend using the following author in this commit, I
> hope there are commit rights:
> astyle-kdelibs <scri...@kde.org>
Please don't abuse scripty for this.
Simply commit the re-styling of the code as yourself, it won't pose any issue.
>
> @Ben?
>
> Similar for other scripts.
Regards,
Ben
>
> On 4 March 2015 at 12:23, Friedrich W. H. Kossebau <kosse...@kde.org> wrote:
>> Am Mittwoch, 4. März 2015, 00:10:28 schrieb Friedrich W. H. Kossebau:
>>> Am Dienstag, 3. März 2015, 23:37:02 schrieb Jaroslaw Staniek:
>>> > First let's verify if astyle-kdelibs changes only whitespace.
>>>
>>> Okay, I propose that we wait with the porting start until this got sorted
>>> out.
>>>
>>> No pressing need to really start porting today, 2-3 more days don't hurt
>>> here, and there are enough bugs to fix for 2.9. Better have all maintainers
>>> aligned in the goals.
>>> Most people have more time only at the WE anyway :)
>>>
>>> Kicked off a qt5 build for the night to hopefully get that "normalize" tool,
>>> might give a diff some testing tomorrow afternoon.
>>
>> I found meanwhile that the normalize tool can be compiled on its own, is not
>> part of the actual Qt build. So I did, and with the patched astyle ran
>> astyle-
>> kdelibs on this morning's Calligra master.
>>
>> Please see the result committed as branch for preview to the central repo,
>> with 3rd-party, libs/{db,koreport,koproperty}, plugins/reporting not touched,
>> (because either going to be removed or 3rd-party):
>>
>> astyle-kdelibs-result-2015-03-04-kossebau
>>
>> Would those changes work for everyone? Please give feedback as quickly as
>> possible, so you are heard.
>> (The branch will be deleted later on, just used for convenience to have
>> everyone see the possible changes.)
>>
>> Cheers
>> Friedrich
>>
>> _______________________________________________
>> Kexi-devel mailing list
>> kexi-de...@kde.org
>> https://mail.kde.org/mailman/listinfo/kexi-devel
>
>
>
> --
> regards, Jaroslaw Staniek
>
> KDE:
> : A world-wide network of software engineers, artists, writers, translators
> : and facilitators committed to Free Software development - http://kde.org
> Calligra Suite:
> : A graphic art and office suite - http://calligra.org
> Kexi:
> : A visual database apps builder - http://calligra.org/kexi
> Qt Certified Specialist:
> : http://www.linkedin.com/in/jstaniek
_______________________________________________
calligra-devel mailing list
calligra-devel@kde.org
https://mail.kde.org/mailman/listinfo/calligra-devel
