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

Clarify that empty bars are not permitted. #137

Open
ahankinson opened this issue Jun 13, 2024 · 4 comments
Open

Clarify that empty bars are not permitted. #137

ahankinson opened this issue Jun 13, 2024 · 4 comments
Assignees
Labels
Type: Spec Enhancement New, non-normative changes (examples, elaborations, etc)

Comments

@ahankinson
Copy link
Contributor

e.g., You can't do //////

@ahankinson ahankinson self-assigned this Jun 13, 2024
@BaMikusi
Copy link
Collaborator

But please make an exception for the double bar line.

@ahankinson
Copy link
Contributor Author

Could you clarify that?

We just chatted about this on our morning call, and noted that the above could be interpreted as either 6 singles, or 3 doubles. Two slashes will obviously be allowed, but three + will not, without any intervening characters.

@lpugin lpugin added the Type: Spec Enhancement New, non-normative changes (examples, elaborations, etc) label Jun 13, 2024
@BaMikusi
Copy link
Collaborator

I really just meant to point out that // must be OK in the code string, since that's the double bar line and not an empty bar (from the minimalist title and desciption it was not obvious whether this had been considered).

@lpugin
Copy link
Contributor

lpugin commented Jun 13, 2024

It also think that the sentence

Bar lines SHOULD be inserted at intervals that correspond to the current time signature. 

is out of scope for the specs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Spec Enhancement New, non-normative changes (examples, elaborations, etc)
Projects
None yet
Development

No branches or pull requests

3 participants