Skip to content

Commit 227092e

Browse files
Java: Minor corrections to comments
Co-authored-by: Marcono1234 <Marcono1234@users.noreply.github.com>
1 parent 3ef9498 commit 227092e

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

java/ql/src/semmle/code/java/frameworks/guava/StringUtils.qll

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
/** Definitions of flow steps through the various string utility fenctions in the Guava framework. */
1+
/** Definitions of flow steps through the various string utility functions in the Guava framework. */
22

33
import java
44
private import semmle.code.java.dataflow.FlowSteps
@@ -51,9 +51,9 @@ private class GuavaStringsTaintPreservingMethod extends TaintPreservingCallable
5151
GuavaStringsTaintPreservingMethod() {
5252
this.getDeclaringType() instanceof TypeGuavaStrings and
5353
// static String emptyToNull(String string)
54-
// static String emptyToNull(String string)
55-
// static String padEnd(String string, int minLength, char padChar)
54+
// static String nullToEmpty(String string)
5655
// static String padStart(String string, int minLength, char padChar)
56+
// static String padEnd(String string, int minLength, char padChar)
5757
// static String repeat(String string, int count)
5858
// static String lenientFormat(String template, Object ... args)
5959
this.hasName(["emptyToNull", "nullToEmpty", "padStart", "padEnd", "repeat", "lenientFormat"])
@@ -86,6 +86,7 @@ private class GuavaJoinerBuilderMethod extends GuavaJoinerMethod, TaintPreservin
8686
// static Joiner on(String separator)
8787
// Joiner skipNulls()
8888
// Joiner useForNull(String nullText)
89+
// Joiner.MapJoiner withKeyValueSeparator(char keyValueSeparator)
8990
// Joiner.MapJoiner withKeyValueSeparator(String keyValueSeparator)
9091
// Joiner.MapJoiner useForNull(String nullText) [on MapJoiner]
9192
this.hasName(["on", "skipNulls", "useForNull", "withKeyValueSeparator"])

0 commit comments

Comments
 (0)