garydgregory commented on PR #182:
URL: 
https://github.com/apache/commons-configuration/pull/182#issuecomment-1133644573

   > @garydgregory, do you have any objections to this being merged?
   
   It might be ok but the single variable check will fail if a variable default 
value contains the end marker (escaped). Maybe the isSingleVariable method 
should document that this is only a simple attempt at optimization?
   
   The larger issue is that we have interpolation code all over in Apache 
projects. I'm trying to standardize on the Commons Text implementation but that 
implementation is not perfect either...


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