search

LibraryOfCongress / bagit-spec

8 stars 7 forks source link

Consistent "line feed (LF)" instead of "newline" #26

Closed stain closed 6 years ago

stain commented 6 years ago

As agreed in https://github.com/LibraryOfCongress/bagit-spec/pull/17#discussion_r184976972 to match the Unicode standards we should use "line feed" to refer to the LF character, rather than "newline".

Note that I also fixed one error in line 528 where CR/LF had been swapped.

stain commented 6 years ago

As this text is quite repetitive with LF, CR or CRLF definition every time, should we rather define this at the top somewhere and reference it as "newline characters"?

Or perhaps it's better to stay explicit every time so people don't think it's just \n.

johnscancella commented 6 years ago

I personally like to be explicit exactly for the reason that people could think it is just \n