Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upTreatment of `length` is inconsistent in 9.4.2 and 9.4.3 #1236
Comments
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment
Hide comment
|
Seems fine. |
ljharb
added
editorial change
good first patch
labels
Jun 18, 2018
rkirsling
referenced this issue
Jun 19, 2018
Merged
Editorial: "length" should be used consistently #1239
ljharb
closed this
in
#1239
Aug 9, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
rkirsling commentedJun 18, 2018
9.4.2 has a long first paragraph describing the
lengthproperty:Yet it also has the following sentence dangling as its own paragraph afterwards:
Seems like the latter should be deleted and the word "non-configurable" added to the former.
9.4.3 exclusively uses
"length"in its exposition:This is locally consistent, but I think it could be changed to just
lengthfor consistency with 9.4.2.Note: The string
"length"is used for good reason in the algorithmic steps; this issue is purely about the explanatory text.