[jira] [Commented] (ARTEMIS-5528) Replace SimpleString.containsEitherOf with WildcardConfiguration.isWild

["ASF subversion and git services (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ARTEMIS-5528?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17959536#comment-17959536 ] 

ASF subversion and git services commented on ARTEMIS-5528:
----------------------------------------------------------

Commit 4cb358f3f9c097c5b1fc6fe1a9d298ce4e064158 in activemq-artemis's branch refs/heads/main from Justin Bertram
[ https://gitbox.apache.org/repos/asf?p=activemq-artemis.git;h=4cb358f3f9 ]

ARTEMIS-5528 replace SimpleString.containsEitherOf with WildcardConfiguration.isWild


> Replace SimpleString.containsEitherOf with WildcardConfiguration.isWild
> -----------------------------------------------------------------------
>
>                 Key: ARTEMIS-5528
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-5528
>             Project: ActiveMQ Artemis
>          Issue Type: Task
>            Reporter: Justin Bertram
>            Assignee: Justin Bertram
>            Priority: Minor
>              Labels: pull-request-available
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> The {{containsEitherOf}} method in {{SimpleString}} is a general purpose method to check if a {{SimpleString}} contains either of two {{char}}. However, this method is only ever used to check if the {{SimpleString}} contains wildcard characters. This job is better suited for the {{isWild}} method of {{WildcardConfiguration}}. Replacing the former with the latter will be more clear and remove unnecessary code.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@activemq.apache.org
For additional commands, e-mail: issues-help@activemq.apache.org
For further information, visit: https://activemq.apache.org/contact