#1475 Use fixed character array for C0 and C1 control character name list

[Zufu Liu]

in Editor::SetRepresentations() and ControlCharacterString(), const char *const reps[] can be changed to static constexpr char reps[][4].

with the change Scintilla.dll is 1KiB smaller.

[Neil Hodgson]

To me, adding bounds makes the code more fragile and difficult to modify.

[Zufu Liu]

These name are standardized and will not changed in the further, if the bounds is smaller (e.g. not including NUL terminator) there is a compiler error.

[Neil Hodgson]

May be sensible to share the C0 strings between SetRepresentations and ControlCharacterString and extract these to another module. Either in PositionCache as that is where SpecialRepresentations currently lives or into a new Representations module.

Something like the attached patch.

[Zufu Liu]

Adding SetDefaultRepresentations() looks good to me.

[Neil Hodgson]

Moved default representation code into SpecialRepresentations class with [7cf62b].

[Neil Hodgson]

The original feature won't be included.