Wireshark-commits: [Wireshark-commits] master-1.10 9aed0cf: Use g_ascii_isalnum() rather than isaln
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Thu, 16 Oct 2014 23:54:44 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=9aed0cffbd78e2084ff033cb634dd73a097c1325
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-1.10
Repository: wireshark

Commits:

9aed0cf by Guy Harris (guy@xxxxxxxxxxxx):

    Use g_ascii_isalnum() rather than isalnum().
    
    That way, we don't have to worry about casting the argument (which, in
    one place, was done wrong - casting to int preserves the sign-extension
    done with signed chars), and don't have to worry about a locale in which
    particular 8-bit byte values are considered alphanumeric characters.
    
    Change-Id: I129b4bfdad70ade4ab6e0a1d2c13d59ae9e6f524
    Reviewed-on: https://code.wireshark.org/review/4751
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    (cherry picked from commit e6fcff1eb62b581eb7fb3d7af1d2e38612a00b30)
    Reviewed-on: https://code.wireshark.org/review/4753
    

Actions performed:

    from  1916b57   Make sort_alphabet() not fail too badly with characters in [128,255].
    adds  9aed0cf   Use g_ascii_isalnum() rather than isalnum().


Summary of changes:
 epan/dfilter/dfilter-macro.c |    5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)