[Mono-dev] revert of r104029/30/31

Gert Driesen gert.driesen at telenet.be
Mon May 26 05:10:27 EDT 2008


--------------------------------------------------
From: "Sebastien Pouliot" <sebastien at ximian.com>
Sent: Monday, May 26, 2008 1:21 AM
To: <gert.driesen at pandora.be>
Cc: "mono-devel-list" <mono-devel-list at lists.ximian.com>
Subject: [Mono-dev] revert of r104029/30/31

> Gert,
>
> Please *post* your patches for review *before* committing them!
> Those changes were obviously untested since they broke the unit tests.

Sorry about that. I was too confident that these changes were harmless, my 
mistake.

I'll look into this, and - if you want - post an updated patch for review.

> They also mix formatting changes (not all in respect of mono style) with
> code changes (parameter names) making this harder, than it has to be, to
> review.

I'm pretty sure all the formatting changes respect the "mono style", but 
feel free to correct me.

I know we had a "discussion" on whether braces for the class/namespace 
should go on a separate line or not, and I know you thought they should go 
on the same line. However, both the mono guidelines and several committers 
on #mono confirmed that they should go on a separate line.

I don't think I want to spend an hour or two splitting up the argument name 
changes with the formatting changes, so if you prefer to keep the sources in 
the state they were before my changes then I'll just revert my local 
changes.

My formatting changes were limited to:

* changes loads of spaces to tabs
* removing loads of trailing tabs
* moving braces on class/namespace declaration to separate line (probably 
less than 2% of my changes)

> Also note that such *big* changes always cause a lot of conflicts for
> the code maintainer(s), since they often have uncommitted code or patch
> in testing. So a bit of warning before committing is a *minimum*.

True.

Gert 




More information about the Mono-devel-list mailing list