Gary,

I agree with you it’s a bug. I would not expect that behavior from that method. 

Mello 

> On Jun 30, 2020, at 8:25 PM, Gary Gregory <[email protected]> wrote:
> 
> Hi All:
> 
> I think we might have a bug in StringSubstitutor exemplified in a comment I
> just added in:
> 
> org.apache.commons.text.StringSubstitutorTest.testReplaceWeirdPattens()
> 
>        doTestNoReplace("${");
>        // TODO this looks like a bug since a $ is removed but this is not
> a variable.
>        // doTestNoReplace("$${");
>        // doTestNoReplace("$${a");
>        // doTestNoReplace("$$${");
>        // doTestNoReplace("$$${a");
> 
> I thought that we would only strip $ in front of a _complete_ variable, not
> anywhere in front of a variable _prefix_.
> 
> Granted this is an edge case but still...
> 
> Any thoughts?
> 
> Gary

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to