Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

empty reporter and boolean blocks are invalid #93

Open
pdawyndt opened this issue Aug 3, 2018 · 0 comments
Open

empty reporter and boolean blocks are invalid #93

pdawyndt opened this issue Aug 3, 2018 · 0 comments
Labels
grammar This issue requires changes to the grammar
Milestone

Comments

@pdawyndt
Copy link

pdawyndt commented Aug 3, 2018

A warning should be issued when using an empty reporter block (()) or an an empty boolean block (<>) since Blockly does not allow this; it automatically assignes names to these blocks ((a), (b), ...). The logic should than be that

say {}

generates a say block without a shadow (@Ellen102 can you add a graphical example of this) and

say {""} 

or

say "" 

generates a say block with a shadow; the options

say {()}
say ()
say {<>}
say <> 

are illigal and should issue a warning, because empty reporters and booleans are not supported because of the automatic naming of Google Blockly; as a recovery a block with a %1 placeholder be issued

@Ellen102 Ellen102 added this to the Future milestone Aug 31, 2018
@Ellen102 Ellen102 added the grammar This issue requires changes to the grammar label Aug 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
grammar This issue requires changes to the grammar
Projects
None yet
Development

No branches or pull requests

2 participants