Wireshark-commits: [Wireshark-commits] master 0d9b449: 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: Fri, 17 Oct 2014 01:26:30 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=0d9b449ae95f26e86f8b433c7f4f99341443b048
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

0d9b449 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>
    
    Change-Id: I1384e3ae4ccba6af9220a08773abcde7ea320573
    Reviewed-on: https://code.wireshark.org/review/4755
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  359a32f   No need for our own wrappers around tolower() and toupper().
    adds  0d9b449   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(-)