Wireshark-commits: [Wireshark-commits] master f542ab8: file_util.h includes io.h, we don't need to
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, 6 Nov 2015 23:44:19 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=f542ab8e8ef5919fd9c2d4bbbf05ae0d7dc7e9d1
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

f542ab8 by Guy Harris (guy@xxxxxxxxxxxx):

    file_util.h includes io.h, we don't need to do so ourselves.
    
    While we're at it, get rid of a commented-out include, and update a
    comment to give more information.
    
    Change-Id: I910a26a3d7f4a50f0559abe5f2ab97a93c80357f
    Reviewed-on: https://code.wireshark.org/review/11610
    Petri-Dish: Guy Harris <guy@xxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  161f449c  4294967296 is not guaranteed to be a valid integer constant.
    adds  f542ab8   file_util.h includes io.h, we don't need to do so ourselves.


Summary of changes:
 wsutil/file_util.c |   39 ++++++++++++++++++++++++++++++++-------
 1 file changed, 32 insertions(+), 7 deletions(-)