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

corner-case for ''' strings #7

Open
florianschanda opened this issue Feb 15, 2023 · 0 comments
Open

corner-case for ''' strings #7

florianschanda opened this issue Feb 15, 2023 · 0 comments
Labels
good first issue Fairly self-contained issue that is ideal as a first contribution topic: core Affects lexer/parser/infrastructure topic: LRM Affects the Language Reference Manual

Comments

@florianschanda
Copy link
Collaborator

In the LRM we have something like this:

  bnf = '''
    foo ::= bar
            { wibble }
  '''

This comes out as

foo ::= bar
{ wibble }

It would be better if it came out like this:

foo ::= bar
        { wibble }

We could change the rules to only ignore absolute first line (i.e. the stuff on the same line as the ''' when figuring out common whitespace). But still ignore blank lines. But this is rapidly becoming a mess.

@florianschanda florianschanda added the topic: LRM Affects the Language Reference Manual label Feb 15, 2023
@florianschanda florianschanda added the topic: core Affects lexer/parser/infrastructure label Jul 17, 2023
@florianschanda florianschanda added the good first issue Fairly self-contained issue that is ideal as a first contribution label Oct 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Fairly self-contained issue that is ideal as a first contribution topic: core Affects lexer/parser/infrastructure topic: LRM Affects the Language Reference Manual
Projects
None yet
Development

No branches or pull requests

1 participant