add missing eol at eof (POSIX compliance) #1690

New Issue

MrUnknownDE · 2026-04-06T02:13:58+02:00

MrUnknownDE commented

2026-04-06 02:13:58 +02:00

Originally created by @tessus on 3/24/2023

According to the POSIX standard a line is defined as follows:

3.206 Line

A sequence of zero or more non-<newline> characters plus a terminating <newline> character.

/ref https://pubs.opengroup.org/onlinepubs/9699919799/basedefs/V1_chap03.html#tag_03_206

I am very sorry for the nitpicking, but this really drives me nuts. I never understood why people started to omit the last EOL. At university (almost 30 years ago) we had a system's programming course and it was drilled into us that a line without an EOL (newline) is not a line. Additionally many compilers complained, others threw an error.

I do understand that there is this unwritten rule that for minimized js there shouldn't be an EOL. IMO this is also a mistake, but who knows - that last byte could throw the world into chaos, if it had to be transferred as well.

I hope you don't see me as a total nut case.

*Originally created by @tessus on 3/24/2023* According to the POSIX standard a line is defined as follows: ``` 3.206 Line A sequence of zero or more non-<newline> characters plus a terminating <newline> character. ``` /ref https://pubs.opengroup.org/onlinepubs/9699919799/basedefs/V1_chap03.html#tag_03_206 ---- I am very sorry for the nitpicking, but this really drives me nuts. I never understood why people started to omit the last EOL. At university (almost 30 years ago) we had a system's programming course and it was drilled into us that a line without an EOL (newline) is not a line. Additionally many compilers complained, others threw an error. I do understand that there is this unwritten rule that for minimized js there shouldn't be an EOL. IMO this is also a mistake, but who knows - that last byte could throw the world into chaos, if it had to be transferred as well. I hope you don't see me as a total nut case.

MrUnknownDE closed this issue

2026-04-06 02:13:58 +02:00

Sign in to join this conversation.

No Label

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github/vaultwarden#1690