elharo opened a new pull request, #68:
URL: https://github.com/apache/maven-mapping/pull/68

   ## Summary
   
   Remove the unnecessary empty capture group and  prefix trick from the  
pattern in .
   
   ## Problem
   
   The previous regex used  and  with , creating the pattern:
   
   
   
   This contained an unused optional group  that served no real purpose — the  
string was a dummy prefix intended to be filtered out by the optional group, 
but since the custom prefix/suffix already delimit  expressions, the prefix 
filtering is redundant.
   
   ## Fix
   
   The new approach uses an empty  with the full capture pattern in  and passes 
 for :
   
   
   
   This produces the simpler pattern  with the key captured directly in group 1.
   
   ## Verification
   
   All existing tests pass without modification:
   -  
   -  
   -  
   -  
   -  
   -  (MWAR-212) 
   
   Closes #67


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to