Wireshark-commits: [Wireshark-commits] master-1.10 fc8132c: Don't use <ctype.h> macros.

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Mon, 20 Oct 2014 08:59:04 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=fc8132ca84684e80ae086bba005433f919d6afd6
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-1.10
Repository: wireshark

Commits:

fc8132c by Guy Harris (guy@xxxxxxxxxxxx):

    Don't use <ctype.h> macros.
    
    They're locale-dependent (although those *particular* ones might not
    be).
    
    Remove no-longer necessary (or not-even-necessary-before-this-change)
    includes of <ctype.h>
    
    (cherry picked from commit 0d9b449ae95f26e86f8b433c7f4f99341443b048)
    
    Change-Id: Ic2be240369c62a06bd5152fda46e6ad9027570cf
    Reviewed-on: https://code.wireshark.org/review/4869
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  68ff8fb   Get rid of another toupper() call in favor of g_ascii_toupper().
    adds  fc8132c   Don't use <ctype.h> macros.


Summary of changes:
 epan/ftypes/ftype-bytes.c  |    1 -
 epan/ftypes/ftype-guid.c   |    3 +--
 epan/ftypes/ftype-string.c |    1 -
 epan/ftypes/ftype-time.c   |    5 ++---
 4 files changed, 3 insertions(+), 7 deletions(-)