Make empty @return statement a parser error

mgreter · mgreter · commit c522143aad37 · 2015-07-26T17:05:10.000+02:00
diff --git a/src/parser.cpp b/src/parser.cpp
@@ -2128,6 +2128,9 @@ namespace Sass {
 
   Return* Parser::parse_return_directive()
   {
+    // check that we do not have an empty list (ToDo: check if we got all cases)
+    if (peek_css < alternatives < exactly < ';' >, exactly < '}' >, end_of_file > >())
+    { css_error("Invalid CSS", " after ", ": expected expression (e.g. 1px, bold), was "); }
     return new (ctx.mem) Return(pstate, parse_list());
   }
 

Original file line number	Diff line number	Diff line change
`@@ -2128,6 +2128,9 @@ namespace Sass {`
`2128`	`2128`
`2129`	`2129`	`Return* Parser::parse_return_directive()`
`2130`	`2130`	`{`
	`2131`	`+ // check that we do not have an empty list (ToDo: check if we got all cases)`
	`2132`	`+ if (peek_css < alternatives < exactly < ';' >, exactly < '}' >, end_of_file > >())`
	`2133`	`+ { css_error("Invalid CSS", " after ", ": expected expression (e.g. 1px, bold), was "); }`
`2131`	`2134`	`return new (ctx.mem) Return(pstate, parse_list());`
`2132`	`2135`	`}`
`2133`	`2136`