Re: [ jEdit-devel ] Suggested fix for bug 127389 (JStandard plugin)

[Brian C. <bca...@ya...>]

My bad.  I didn't realize $ bound that tightly.  The
fixed line should then be:

public final String REG_EXP_FINAL_FIELD =
"^(([A-Z_]+)|serialVersionUID)$";

Thanks,
Brian

--- mike dillon <md...@st...> wrote:
> On Thu, 22 Feb 2001, Brian Carothers wrote:
> > Change line 35 in jstandard.JStandardRules.java
> from:
> >     public final String REG_EXP_FINAL_FIELD =
> > "^[A-Z_]+$";
> > to:
> >     public final String REG_EXP_FINAL_FIELD =
> "^([A-Z_]+)|serialVersionUID$";
> 
> you have given here a regexp... i do not think it
> means what you think means:
> 
> 	Either match at string start one or more "word"
> characters, capturing
> 	them as a group, OR match the literal string
> "serialVersionUID", ending
> 	at the last position in the string.
> 
> you probably meant:
> 	/^([A-Z_]+|serialVersionUID)$/
> 
> or:
> 	/^(([A-Z_]+)|serialVersionUID)$/
> 
> -md



__________________________________________________
Do You Yahoo!?
Yahoo! Auctions - Buy the things you want at great prices! http://auctions.yahoo.com/