Hi, On Sun, 1 Jun 2014 15:03:20 +0900 Nobuhiro Ban <[email protected]> wrote: > >+ protected static final Pattern CLASS_ACCESS_PATTERN = Pattern > >+ .compile("(.*\\.|^|.*|\\[('|\"))class(\\.|('|\")]|\\[).*", > >+ Pattern.CASE_INSENSITIVE); > > It's very strange regexp. Because we know (P1|.*|P2) == .* . > This pattern will match to words other than "class", eg. "fooClass".
Any class should be accepted, maybe it'd cause some trouble but non-class should not named as *class, IMHO. -- Hideki Yamane <[email protected]> -- To UNSUBSCRIBE, email to [email protected] with a subject of "unsubscribe". Trouble? Contact [email protected]
