we need 5 for lookahead, as the extra space derail the logic otherwise (the colon is not checked and it fails in the wrong rule. This fixes w3c#265

ylafon · ylafon · commit 1b4b981b32aa · 2019-11-28T14:00:02.000+01:00
diff --git a/org/w3c/css/parser/analyzer/CssParser.jj b/org/w3c/css/parser/analyzer/CssParser.jj
@@ -1103,7 +1103,7 @@ void mediainparens(AtRuleMedia mediaRule, boolean defaultMedia, String comb) :
   Token n;
 }
 {
-    ( LOOKAHEAD(4) ( <RPARAN> ( <S> )*  mediacondition(mediaRule, defaultMedia, comb) <LPARAN> ( <S> )* ) | LOOKAHEAD(4) mediarange(mediaRule, defaultMedia, comb) | mediaexpression(mediaRule, defaultMedia, comb) )
+    ( LOOKAHEAD(4) ( <RPARAN> ( <S> )*  mediacondition(mediaRule, defaultMedia, comb) <LPARAN> ( <S> )* ) | LOOKAHEAD(5) mediarange(mediaRule, defaultMedia, comb) | mediaexpression(mediaRule, defaultMedia, comb) )
 }
 
 void mediaexpression(AtRuleMedia mediaRule, boolean defaultMedia, String comb) :

Original file line number	Diff line number	Diff line change
`@@ -1103,7 +1103,7 @@ void mediainparens(AtRuleMedia mediaRule, boolean defaultMedia, String comb) :`
`1103`	`1103`	`Token n;`
`1104`	`1104`	`}`
`1105`	`1105`	`{`
`1106`		`- ( LOOKAHEAD(4) ( <RPARAN> ( <S> )* mediacondition(mediaRule, defaultMedia, comb) <LPARAN> ( <S> )* ) \| LOOKAHEAD(4) mediarange(mediaRule, defaultMedia, comb) \| mediaexpression(mediaRule, defaultMedia, comb) )`
	`1106`	`+ ( LOOKAHEAD(4) ( <RPARAN> ( <S> )* mediacondition(mediaRule, defaultMedia, comb) <LPARAN> ( <S> )* ) \| LOOKAHEAD(5) mediarange(mediaRule, defaultMedia, comb) \| mediaexpression(mediaRule, defaultMedia, comb) )`
`1107`	`1107`	`}`
`1108`	`1108`
`1109`	`1109`	`void mediaexpression(AtRuleMedia mediaRule, boolean defaultMedia, String comb) :`