[
https://issues.apache.org/jira/browse/ARTEMIS-5528?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Justin Bertram updated ARTEMIS-5528:
------------------------------------
Description: 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. (was: The
{{containsEitherOf}} method in {{SimpleString}} is a general purpose method to
check if a {{SimpleString}} contains either of two {{char}}s. 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.)
> 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: Major
>
> 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: [email protected]
For additional commands, e-mail: [email protected]
For further information, visit: https://activemq.apache.org/contact
