[feaLib] "Expected name id < 255" check

@brawer is it OK to remove this check from feaLib’s parse_nameid_?

        if nameID > 255:
            raise FeatureLibError("Expected name id < 255",
                                  self.cur_token_location_)

I ran into it while working on a project that requires hardcoding some nameids, and I can’t think of a reason for blocking them, specially since they’re outside the reserved range.

[miguelsousa]

@Arno-Enslin https://github.com/fonttools/fonttools/commit/12b7c60a6817f43557802695c1af9180d1b3c53a

[Arno-Enslin]

From the OT spec:

Name IDs 256 to 32767, inclusive, are reserved for font-specific names such as those referenced by a font’s layout features.

Instead of removing the check, wouldn’t it be better to check, whether the ID is < 32767 (or 32768)?