Skip to content

Commit 33c6e0d

Browse files
committed
no need to return values in pageContent() as it is handled inside the function
1 parent 19bc1a3 commit 33c6e0d

1 file changed

Lines changed: 4 additions & 6 deletions

File tree

org/w3c/css/parser/analyzer/CssParser.jj

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1221,9 +1221,8 @@ void unused_production_definition() :
12211221
*/
12221222
void page() :
12231223
{
1224-
ArrayList<CssProperty> v ;
12251224
Token n = null ;
1226-
ArrayList<CssProperty> collectv = new ArrayList<CssProperty>() ;
1225+
ArrayList<CssProperty> collectv = new ArrayList<CssProperty>();
12271226
CssSelectors s = new CssSelectors(ac);
12281227
AtRule old = getAtRule() ;
12291228
AtRulePage newRule = AtRulePage.getInstance(ac.getCssVersion());
@@ -1236,7 +1235,7 @@ void page() :
12361235
<PAGE_SYM> { if (!isCss1) newAtRule(newRule);
12371236
} ( <S> )*
12381237
( page_selector_list(newRule) )?
1239-
<LBRACE> ( <S> )* ( v=pageContent() ) <RBRACE> ( <S> )*
1238+
<LBRACE> ( <S> )* ( pageContent() ) <RBRACE> ( <S> )*
12401239
{
12411240
if (!isCss1) {
12421241
endOfRule();
@@ -1261,9 +1260,9 @@ void page() :
12611260
}
12621261
}
12631262

1264-
ArrayList<CssProperty> pageContent() :
1263+
void pageContent() :
12651264
{ CssProperty prop;
1266-
ArrayList<CssProperty> v = new ArrayList<CssProperty>();
1265+
ArrayList<CssProperty> v = null;
12671266
ArrayList<CssProperty> allv = new ArrayList<CssProperty>();
12681267
CssSelectors s = new CssSelectors(ac);
12691268
boolean gotContent = false;
@@ -1287,7 +1286,6 @@ ArrayList<CssProperty> pageContent() :
12871286
addProperty(allv);
12881287
handleRule(s, allv);
12891288
}
1290-
return allv;
12911289
}
12921290
}
12931291

0 commit comments

Comments
 (0)