codesaway / regexplus Goto Github PK

Exception in thread "main" java.lang.NullPointerException
at info.codesaway.util.regex.Refactor.anyCondition(Refactor.java:1339)
at info.codesaway.util.regex.Refactor.acceptTestingGroup(Refactor.java:1277)
at info.codesaway.util.regex.Refactor.refactorConditionalPattern(Refactor.java:2611)
at info.codesaway.util.regex.Refactor.refactor(Refactor.java:971)
at info.codesaway.util.regex.Refactor.(Refactor.java:461)
at info.codesaway.util.regex.Refactor.(Refactor.java:421)
at info.codesaway.util.regex.Pattern.compile(Pattern.java:3313)
at info.codesaway.util.regex.Pattern.(Pattern.java:3172)
at info.codesaway.util.regex.Pattern.compile(Pattern.java:1892)
at info.codesaway.util.regex.Pattern.compile(Pattern.java:1862)
at info.codesaway.util.regex.Pattern.compile(Pattern.java:1802)
at info.codesaway.util.RegexTesting.main(RegexTesting.java:10)

For example, Pattern flag UNICODE_CHARACTER_CLASS

This allows \w and other built-in types to work with unicode characters. You can still pass the java.util.regex.Pattern int value, since RegExPlus uses Java's regex engine behind the scenes. However, adding this constant to RegExPlus directly will be much preferred.

For example, some syntax was added to Java's regex, which works in RegExPlus (since under the hood I'm using Java's regex engine). However, my documentation is outdated and doesn't specify the new syntax (such as \h and \v).

Looks like would be the following regex (since \v matches a single line separator)

(?>\r\n|\v)

https://www.pcre.org/original/doc/html/pcrepattern.html

Newline sequences
Outside a character class, by default, the escape sequence \R matches any Unicode newline sequence. In 8-bit non-UTF-8 mode \R is equivalent to the following:

(?>\r\n|\n|\x0b|\f|\r|\x85)

Java 9 added java.util.regex.Matcher$ImmutableMatchResult, which represents the MatchResult. Need to refactor code to handle this.

java.lang.ClassCastException: class java.util.regex.Matcher$ImmutableMatchResult cannot be cast to class java.util.regex.Matcher (java.util.regex.Matcher$ImmutableMatchResult and java.util.regex.Matcher are in module java.base of loader 'bootstrap')
at info.codesaway.util.regex.Matcher.clone(Matcher.java:263)
at info.codesaway.util.regex.Matcher.toMatchResult(Matcher.java:254)

LcsString diff0 method does not correctly create a valid diff in all cases (doesn't show correct number of matches)

		// diff0 has 0 matching characters, yet should be the same number as LCS
		// (this is yielding an invalid diff)
		LcsString lcsString = new LcsString("warningmsg", "naming");
		//		LcsString lcsString = new LcsString("naming", "warningmsg");

		System.out.println("LCS: " + lcsString.lcsLength());
		System.out.println(
				"diff match: " + lcsString.diff().stream().filter(d -> d.getType() == LcsDiffType.NONE).count());
		System.out.println(
				"diff0 match: " + lcsString.diff0().stream().filter(d -> d.getType() == LcsDiffType.NONE).count());

		System.out.println("diff()");
		lcsString.diff().forEach(System.out::println);
		System.out.println();

		System.out.println("diff0()");
		lcsString.diff0().forEach(System.out::println);

• appendReplacement​(StringBuilder sb, String replacement)
• appendTail​(StringBuilder sb)
• replaceAll​(Function<MatchResult,​String> replacer)
• results()
• replaceFirst​(Function<MatchResult,​String> replacer)

Also check Pattern to see what should be added

To improve performance, can use String comparison, versus regex

https://codesaway.info/RegExPlus/named.html

It's hidden in a noscript block for some reason. It should be displayed on the actual page always (might want to reformat though)