Skip to content

Commit e634c8c

Browse files
committed
clearer warning on Parser
1 parent 5b05aa5 commit e634c8c

File tree

1 file changed

+4
-2
lines changed
  • dot-parse/src/main/java/com/google/common/labs/parse

1 file changed

+4
-2
lines changed

dot-parse/src/main/java/com/google/common/labs/parse/Parser.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -70,8 +70,10 @@
7070
* But it's more efficient to factor out common left prefix. For example instead of {@code
7171
* anyOf(expr.followedBy(";"), expr)}, use {@code expr.optionallyFollowedBy(";")} instead.
7272
*
73-
* <p>WARNING: A poorly-written grammar with long common prefixes may incur expensive backtracking
74-
* overhead. And if you define recursive grammars using {@link #define define()} or {@link
73+
* <p>WARNING: A poorly-written grammar with long common prefixes among {@code anyOf()} choices
74+
* may incur expensive backtracking overhead.
75+
*
76+
* <p>WARNING: If you define recursive grammars using {@link #define define()} or {@link
7577
* Parser.Rule}, maliciously crafted input (think of 10K left parens in an expression parser)
7678
* can cause StackOverflowError.
7779
*/

0 commit comments

Comments
 (0)