[
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)