erwindon commented on code in PR #4312:
URL: https://github.com/apache/activemq-artemis/pull/4312#discussion_r1048194987
##########
artemis-commons/src/main/java/org/apache/activemq/artemis/utils/ByteUtil.java:
##########
@@ -39,9 +39,9 @@ public class ByteUtil {
private static final String prefix = "^\\s*(\\d+)\\s*";
private static final String suffix = "(b)?\\s*$";
private static final Pattern ONE = Pattern.compile(prefix + suffix,
Pattern.CASE_INSENSITIVE);
- private static final Pattern KILO = Pattern.compile(prefix + "k" + suffix,
Pattern.CASE_INSENSITIVE);
- private static final Pattern MEGA = Pattern.compile(prefix + "m" + suffix,
Pattern.CASE_INSENSITIVE);
- private static final Pattern GIGA = Pattern.compile(prefix + "g" + suffix,
Pattern.CASE_INSENSITIVE);
+ private static final Pattern KILO = Pattern.compile(prefix + "ki?" +
suffix, Pattern.CASE_INSENSITIVE);
Review Comment:
@michaelandrepearce, that was strictly my intention. the unit test that I
added verifies that. how does this break anything?
--
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]
