hohn/codeql
1
0
Fork 0
mirror of https://github.com/github/codeql.git synced 2026-04-24 00:05:14 +02:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #19560 from owen-mc/java/add-stringreplaceallwithnonregex-test

Browse Source 
Java: Add test showing correct usage
This commit is contained in:
Owen Mansel-Chan
2025-05-22 15:07:58 +01:00
committed by GitHub
parent 4995137145 79453cc103
commit 8b68d95231
2 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
java/ql/src/Performance/StringReplaceAllWithNonRegex.md
View File

@@ -18,6 +18,7 @@ public class Test {
String s1 = "test";
s1 = s1.replaceAll("t", "x"); // NON_COMPLIANT
s1 = s1.replaceAll(".*", "x"); // COMPLIANT
s1 = s1.replace("t", "x"); // COMPLIANT
}
}

1
java/ql/test/query-tests/StringReplaceAllWithNonRegex/Test.java
View File

@@ -3,5 +3,6 @@ public class Test {
String s1 = "test";
s1 = s1.replaceAll("t", "x"); // $ Alert // NON_COMPLIANT
s1 = s1.replaceAll(".*", "x"); // COMPLIANT
s1 = s1.replace("t", "x"); // COMPLIANT
}
}