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-rc-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to