james-server-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Siva" <s...@sivasundaram.com>
Subject Re: cvs commit: jakarta-james/src/java/org/apache/james/transport/mailets RemoteDelivery.java
Date Tue, 12 Nov 2002 19:42:56 GMT
+1.Mixing a patch with code reformatting is always
a head ache.

Its better to have a seperate version for code formatting

----- Original Message -----
From: "Noel J. Bergman" <noel@devtech.com>
To: "James Developers List" <james-dev@jakarta.apache.org>
Sent: Tuesday, November 12, 2002 12:58 AM
Subject: RE: cvs commit:
jakarta-james/src/java/org/apache/james/transport/mailets
RemoteDelivery.java


> I was off-line, but I'd have given a +1 to fixing the bug.  I'll review
the
> patch using the color coding from viewcvs, where I can read the patch in
> context.
>
> However, I do have a request.  When patching bugs, especially during a
code
> freeze period, can we try to avoid reformatting code?  I'd rather see JUST
> the patch, and not the "while I'm at it" cosmetic changes.  We can do the
> cosmetic changes in a separate patch, or after release.  Fair enough?  :-)
>
> --- Noel
>


--
To unsubscribe, e-mail:   <mailto:james-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:james-dev-help@jakarta.apache.org>


Mime
View raw message