Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

myesc() function fixes #195

Closed
wants to merge 2 commits into from
Closed

myesc() function fixes #195

wants to merge 2 commits into from

Commits on Apr 10, 2017

  1. Obsolete htoui() and otoui(); use strtoul(). 

    No sense to keep these two function when libc's strtoul() can do the
same job, but better.
    @Explorer09
    Explorer09 committed Apr 10, 2017
    Configuration menu
    Copy the full SHA
    910efc9 View commit details
    Browse the repository at this point in the history

  2. Fix myesc() 'sptr' conditionals 

    * Don't call isascii() here. It's deprecated in POSIX and not needed
  for myesc's case.
* The check of the character class and range here should match what's
  defined as {ESCSEQ} in scan.l, so for [[:xdigit:]] we use isxdigit();
  for [0-7] we check '0' <= c <= '7' (not isdigit(c) because isdigit is
  locale-dependant in standard's sense)
* Add missing length limit for "\x<hex>" (<hex> is at most 2 digits)
    @Explorer09
    Explorer09 committed Apr 10, 2017
    Configuration menu
    Copy the full SHA
    7907f1d View commit details
    Browse the repository at this point in the history