10 commits

Author	SHA1	Message	Date
ridiculousfish	070a43989f	Mark a null variable const in builtin_echo	2018-10-01 09:47:45 -07:00
ridiculousfish	d7b2576fb5	Mark short_options as const everywhere ... These were needlessly mutable file-scoped variables.	2018-09-29 00:18:36 -04:00
Mahmoud Al-Qudsi	dcced5f1bc	Replace \e with \x1B, as the former is a gcc extension ... While supported by gcc and clang, \e is a gcc-specific extension and not formally defined in the C or C++ standards. See [0] for a list of valid escapes. [0]: https://stackoverflow.com/a/10220539/17027	2018-06-18 00:01:47 -05:00
Kurtis Rader	1e6bd2d968	use the UNUSED(); pattern for the prev change	2017-07-09 19:55:04 -07:00
ridiculousfish	a4eec25447	Fix unused varibale warning	2017-07-09 18:17:20 -07:00
Kurtis Rader	ec1e9ac98d	standardize on builtin_missing_argument()	2017-07-01 14:03:47 -07:00
Kurtis Rader	dd191a9d4b	more wgetopt_long() normalization	2017-06-29 21:49:57 -07:00
Kurtis Rader	82f5fb507d	fix echo -h ... In addition to fixing `echo -h` this includes some debugging related cleanups I made while investigating the issue. Fixes #4120	2017-06-18 22:10:19 -07:00
Kurtis Rader	37b8cfaeba	avoid struct name clashes ... Running the tests on travis revealed that some compilers (or at least with some options) call the wrong struct constructor if there is more than one struct with the same name but differing definitions.	2017-06-16 21:01:57 -07:00
Kurtis Rader	1d4a0fb091	split builtin echo into its own module	2017-06-16 21:01:56 -07:00