[ 
https://issues.apache.org/jira/browse/WW-4489?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14487234#comment-14487234
 ] 

Jasper Rosenberg edited comment on WW-4489 at 4/9/15 11:58 AM:
---------------------------------------------------------------

Well, all I was trying to point out is that the current pattern matches 
anything before {noformat}\\bclass{noformat} because of the "|.*|" section.  I 
figured that wasn't intentional, but wasn't sure what the intent was.


was (Author: perfnorm):
Well, all I was trying to point out is that the current pattern matches 
anything before \\bclass, because of the "|.*|" section.  I figured that wasn't 
intentional, but wasn't sure what the intent was.

> Simplify DefaultExcludedPatternsChecker class pattern
> -----------------------------------------------------
>
>                 Key: WW-4489
>                 URL: https://issues.apache.org/jira/browse/WW-4489
>             Project: Struts 2
>          Issue Type: Bug
>          Components: Core Interceptors
>    Affects Versions: 2.3.20
>            Reporter: Jasper Rosenberg
>            Priority: Minor
>             Fix For: 2.3.24
>
>
> The current pattern that matches class expressions looks like:
> {code:java}"(.*\\.|^|.*|\\[('|\"))\\bclass(\\.|('|\")]|\\[).*"{code}
> However, as far as I can tell, this is completely equivalent to:
> {code:java}".*\\bclass(\\.|('|\")]|\\[).*"{code}
> Not sure if this is a bug, or an improvement.  



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to