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

[Spec] Case insensitive header fields #36

Closed
achimmihca opened this issue Oct 9, 2023 · 3 comments
Closed

[Spec] Case insensitive header fields #36

achimmihca opened this issue Oct 9, 2023 · 3 comments
Labels
Rejected rejected by majority Specification affects specification (spec.md)

Comments

@achimmihca
Copy link

Suggestion

Why are header fields in all uppercase? Is this a convention or enforced by the format?

By defining header field names as case insensitive, one could make the format more robust and readable.

The following would then be equivalent

#VIDEOGAP: 1.2
#videogap: 1.2
#videoGap: 1.2
#VideoGap: 1.2

Use case

Use camelCase or PascalCase for header field names, e.g., #MedleyStartBeat

Extra info/examples/attachments

No response

@Baklap4
Copy link
Collaborator

Baklap4 commented Oct 9, 2023

This has always been the case. Not quite sure what the actual reasoning is.

But by defining it to be case-senstive its up to the txt producers and then we might get a mix of, camelCase, PascalCase and UPPERCASE.

I agree with the readabilty and would vouch for PascalCase, but i wouldn't like to see a mix of the beforementioned ones. Stick with one format as this makes it easy to anticipate for :)

@basisbit
Copy link
Member

basisbit commented Oct 9, 2023

I'd consider the different options to all be approximately similar good for the task. Thus it might be better to just keep it the way it has been the past 15 years or so, unless I am missing an important reason.

@achimmihca
Copy link
Author

achimmihca commented Oct 10, 2023

Ok. It is surprisingly consistent at the moment and it's not causing trouble.
I guess better not change a good running system.

So, this issue can be closed as "won't implement"

@marwin89 marwin89 added the Rejected rejected by majority label Oct 10, 2023
@marwin89 marwin89 added the Specification affects specification (spec.md) label Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Rejected rejected by majority Specification affects specification (spec.md)
Projects
Status: Rejected
Development

No branches or pull requests

4 participants