Re: [PATCH] connman: update to 0.78

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Fri, 2011-12-09 at 14:38 -0200, Otavio Salvador wrote:
> On Fri, Dec 9, 2011 at 12:18, Richard Purdie:

>  
>         f) Reorder variables in the file so as to make the diff
>         unreadable and
>           hard to review
>
> This was indeed done and I am sorry but it does seems more easy to
> follow now.

In future please do this as a separate commit which just changes the
order. I will start just outright rejecting patches which combine code
changes and unrelated ordering changes since it makes the patches
unnecessarily hard to review.
 
>         So all things considered, this kind of change is a nightmare
>         to review
>         in this form and the commit message only covers half the
>         changes it
>         makes. We need to do better than this...
> 
> 
> Not easy to accomplish as the difference between meta-oe and oe-core
> on this was huge. It could have been  done better, sure ... and a v2
> patch will come shortly.

Thanks. I think the best way to improve this in future would be to keep
code changes and ordering (or whitespace) changes in separate commits
(and also enable rename detection on the change you send).

Cheers,

Richard