Branch: refs/heads/master
Revision: 0fb9bc6
Author: mppf
Log Message:
Merge pull request #17607 from mppf/fix-formals-starting-comma
Fix lack of error for formals list starting with comma
A formals list starting with a comma should be a syntax error but was compiling. E.g.
proc foo(, x: int) { }
foo(1);
This PR updates the parser to make it a syntax error and adds a test for it.
- [x] full local testing
- [x] commit rebuilt parser
Reviewed by @lydia-duncan - thanks!
Modified Files:
A test/parsing/ferguson/formal-list-starts-comma.chpl
A test/parsing/ferguson/formal-list-starts-comma.good
M compiler/include/bison-chapel.h
M compiler/parser/bison-chapel.cpp
M compiler/parser/chapel.ypp
Compare: https://github.com/chapel-lang/chapel/compare/780594c08153...0fb9bc60c2bd