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

Leading Spaces #27

Open
brainbuz opened this issue Jan 2, 2022 · 2 comments
Open

Leading Spaces #27

brainbuz opened this issue Jan 2, 2022 · 2 comments

Comments

@brainbuz
Copy link
Contributor

brainbuz commented Jan 2, 2022

I don't think it was ever clearly decided in issue #6 if an extra space at the beginning a line would be allowed or not.

Options:

  1. Ignored [Parsers strip leading white space from lines.]
  2. Skipped [Parsers treat a leading white space as an empty line, which effectively makes a leading space equivalent to #.]

I'm fine with either choice, Robla favored 1 in #6. If 2 it should be made an official comment marker.

@robla
Copy link
Contributor

robla commented Jan 24, 2022

I think there's option 3 and option 4 here:
3. Invalid [Parsers must reject these lines]
4. Undefined [Parsers can choose option 1, 2, or 3, but generators should be discouraged from adding leading whitespace]

Option 1 is still my preferred option (I think), given that it seems likely that handcrafted ABIF files in emails would be nice.

@simberaj
Copy link

I am in favor of 1 precisely because of the handcrafted e-mails.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants