Skip to content

Commit e8c2d72

Browse files
Claudenwslachiewicz
authored andcommitted
fix for issue #279
1 parent 5c1b97b commit e8c2d72

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

src/main/java/org/codehaus/plexus/util/SelectorUtils.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ public static boolean matchPatternStart( String pattern, String str, boolean isC
151151

152152
public static boolean isAntPrefixedPattern( String pattern )
153153
{
154-
return pattern.length() > ( ANT_HANDLER_PREFIX.length() + PATTERN_HANDLER_SUFFIX.length() + 1 )
154+
return pattern.length() > ( ANT_HANDLER_PREFIX.length() + PATTERN_HANDLER_SUFFIX.length())
155155
&& pattern.startsWith( ANT_HANDLER_PREFIX ) && pattern.endsWith( PATTERN_HANDLER_SUFFIX );
156156
}
157157

@@ -283,7 +283,7 @@ private static String toOSRelatedPath( String pattern, String separator )
283283

284284
public static boolean isRegexPrefixedPattern( String pattern )
285285
{
286-
return pattern.length() > ( REGEX_HANDLER_PREFIX.length() + PATTERN_HANDLER_SUFFIX.length() + 1 )
286+
return pattern.length() > ( REGEX_HANDLER_PREFIX.length() + PATTERN_HANDLER_SUFFIX.length())
287287
&& pattern.startsWith( REGEX_HANDLER_PREFIX ) && pattern.endsWith( PATTERN_HANDLER_SUFFIX );
288288
}
289289

src/test/java/org/codehaus/plexus/util/SelectorUtilsTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ public void testExtractPattern() {
5050
*/
5151
@Test
5252
public void testIsAntPrefixedPattern() {
53-
assertFalse(SelectorUtils.isAntPrefixedPattern("%ant[A]")); // single char not allowed
53+
assertTrue(SelectorUtils.isAntPrefixedPattern("%ant[A]")); // single char not allowed
5454
assertTrue(SelectorUtils.isAntPrefixedPattern("%ant[AB]"));
5555
assertFalse(SelectorUtils.isAntPrefixedPattern("%ant[]"));
5656
assertFalse(SelectorUtils.isAntPrefixedPattern("*"));
@@ -61,7 +61,7 @@ public void testIsAntPrefixedPattern() {
6161
*/
6262
@Test
6363
public void testIsRegexPrefixedPattern() {
64-
assertFalse(SelectorUtils.isRegexPrefixedPattern("%regex[A]")); // single char not allowed
64+
assertTrue(SelectorUtils.isRegexPrefixedPattern("%regex[A]")); // single char not allowed
6565
assertTrue(SelectorUtils.isRegexPrefixedPattern("%regex[.*]"));
6666
assertFalse(SelectorUtils.isRegexPrefixedPattern("%regex[]"));
6767
assertFalse(SelectorUtils.isRegexPrefixedPattern("*"));

0 commit comments

Comments
 (0)