Hi Bernd,

Just a minor point...

I see a patch was committed to change StateUtils as part of a patch
containing maven-related work [r369027] by "bommel" which I think is
you. However the changes to StateUtils doesn't seem to have anything to
do with maven.

As far as I can see, the actual change made is fine. It removes a
try/catch clause which was simply ignoring the thrown exception, which
is good as far as I'm concerned. And it does some whitespace cleanups.

However I would have preferred to see this as a separate patch, with a
useful associated comment, rather than as part of a commit for
presumably completely unrelated maven stuff.

Cheers,

Simon

Reply via email to