php/php-langspec

"case-sensitive" and "case-insensitive" are easy to mix up

MasterZydra opened this issue · 3 comments

Hi,
I am following the specification to implement a PHP lexer.
I noticed that there is a mix of the wording "not case-sensitive" and "case-insensitive".
Would it make sense to just use "not case-sensitive" so that it is more clear?
In my opinion "case-sensitive" and "case-insensitive" is easier to accidentally mix it up.

Examples:

Keywords are not case-sensitive.

automatically available to all scripts; their values are not fixed and they are case-insensitive:

Using "not case-sensitive" is probably better, but note that this language spec is severely out of date and does not cover any recent changes/additions to the language.

@Girgias But it is still the official language specification? So I would create a pull request for this change.

@Girgias But it is still the official language specification? So I would create a pull request for this change.

This language specification is, AFAIK, unmaintained.

From what I seem to remember from things I've been told, this was mainly created by Facebook to come up with some common ground for their implementation of HHVM to be compatible with PHP.

So this is probably only going to be good for parsing requirements up to PHP 7.0 maybe 7.1, and thus you won't have things like readonly, constructor property promotion, union/intersection/DNF types, etc.