Remove non-standard \e escape

visq / language-c

Source repository for https://hackage.haskell.org/package/language-c

http://visq.github.io/language-c/

Other

87 stars 45 forks source link

Remove non-standard \e escape #58

Closed bendmorris closed 5 years ago

bendmorris commented 5 years ago

\e is commonly supported but nonstandard; compiling with -Wpedantic results a warning: use of non-standard escape character '\e' [-Wpedantic]. The hex/octal versions would be better since they're part of the standard.

visq commented 5 years ago

No benefit in removing support for \e.

bendmorris commented 5 years ago

The benefit is that language-c would no longer generate nonstandard C.

visq commented 5 years ago

Ah yes, thanks for the clarification.