Allan-QLB commented on PR #673: URL: https://github.com/apache/commons-text/pull/673#issuecomment-2894800198
> This doesn't make sense to me. You can provide a custom default `StringLookup` instead of adding the complexity of an additional layer of processing. > There were 2 cases that would fallback to default lookup when resolving variable with a specified prefix, 1) there is no lookup for the prefix. 2）there is a lookup for the prefix but it cannot resolve this variable. This pr allows deal with the 2 cases differently. It is hard to distinguish the 2 cases in default lookup because the prefix is removed before calling default lookup. -- 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: issues-unsubscr...@commons.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
