This is the mail archive of the java-patches@gcc.gnu.org mailing list for the Java project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: PR 5902: Parse errors



Eric Blake writes:

Thank you for looking into this. 

> OK to commit mainline and branch?

I think you should separate fixing the comments and fixing the
PR. Sometimes it's OK to change one or two things related to formating
and/or coding convention compliance (and I certainly abused that
rule), but for this particular case, the fix looks a bit lost.

> -		      /* {E,e} must have seen at list a digit */
> +		      /* {E,e} must have seen at list a digit. */

While you're at it, ^list^least^

I wanted to give it a little more testing, but the patch appears to be
malformed (first line 11, then 28... probably a paste that went
wrong.) Could you please re-post something? Two different patches
would be really convenient.

Thanks!

./A


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]