Wireshark-commits: [Wireshark-commits] master ef444d3: Squelch a compiler warning and get rid of an
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sun, 7 Sep 2014 09:16:28 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ef444d33fa380922c6ee17bd13d12a16e9505f71
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

ef444d3 by Guy Harris (guy@xxxxxxxxxxxx):

    Squelch a compiler warning and get rid of an unnecessary variable.
    
    Just use "sizeof line" for the size of that array; don't have a separate
    variable with the number of elements of the array (which at least is
    equal to the size of the array, as it's an array of char), as that means
    that you have to remember to change both of them.
    
    Then cast "sizeof line" to int, as the second argument to fgets() is an
    int, not a size_t (fgets(), as I remember, existed before size_t).
    
    Change-Id: I3c65774527f4fcd824d7ae39208ab6e8e33eb9b4
    Reviewed-on: https://code.wireshark.org/review/4023
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  419de19   Create subdissector table for RTAC Serial and have dissectors register with it, rather than have the RTAC Serial dissector go find all of the dissectors its interested in.
    adds  ef444d3   Squelch a compiler warning and get rid of an unnecessary variable.


Summary of changes:
 ui/qt/sctp_chunk_statistics_dialog.cpp |    3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)