Hi, On Sun, 1 Jun 2014 15:03:20 +0900 Nobuhiro Ban <ban.nobuh...@gmail.com> 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 <henr...@debian.or.jp> -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org