Wireshark-commits: [Wireshark-commits] master-1.12 2f585cf: Don't use ctype.h routines.
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Fri, 17 Oct 2014 20:42:40 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=2f585cfc2562f8dd596367185eb661ce65f57806
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-1.12
Repository: wireshark

Commits:

2f585cf by Guy Harris (guy@xxxxxxxxxxxx):

    Don't use ctype.h routines.
    
    That avoids locale dependency and handles possibly-signed chars (which
    we weren't doing before).
    
    (cherry picked from commit 5bfde7c6386b56db80fec85e2ca7dc1d271b461c)
    
    Change-Id: I1faed2959ea2c6868cd27b865f1c3550d7706993
    Reviewed-on: https://code.wireshark.org/review/4782
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  d23160a   isdigit -> g_ascii_isdigit to handle 8-bit bytes correctly.
    adds  2f585cf   Don't use ctype.h routines.


Summary of changes:
 wiretap/catapult_dct2000.c |   27 +++++++++++++--------------
 wiretap/cosine.c           |    3 +--
 wiretap/daintree-sna.c     |   13 ++++++-------
 wiretap/dbs-etherwatch.c   |   19 +++++++++----------
 wiretap/dct3trace.c        |    3 +--
 wiretap/iseries.c          |    5 ++---
 wiretap/netscreen.c        |    5 ++---
 wiretap/vms.c              |   11 +++++------
 8 files changed, 39 insertions(+), 47 deletions(-)