Wireshark-commits: [Wireshark-commits] master-2.6 cfb90bb: Clean up some things.

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Tue, 02 Oct 2018 11:21:16 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=cfb90bbfff2af43d44f5ee7da6f3a496c0c07788
Submitter: "Guy Harris <guy@xxxxxxxxxxxx>"
Changed: branch: master-2.6
Repository: wireshark

Commits:

cfb90bb by Guy Harris (guy@xxxxxxxxxxxx):

    Clean up some things.
    
    Rename ascend_seek() to ascend_find_next_packet(), to indicate what it
    does; it doesn't seek to an arbitrary place, it tries to find the
    starting offset of the next packet when reading sequentially.
    
    Don't have it set the header type - that's the job of the parser.
    
    Don't set the "next packet seek start" when doing random access I/O -
    that field is only for sequential I/O, and we don't want random I/O
    happening at the same time (which can happen in Wireshark) interfering.
    
    Clean up comments.
    
    Change-Id: I2808479eeec074afa16945ffb577b91d8cb356f7
    Reviewed-on: https://code.wireshark.org/review/29975
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    (cherry picked from commit 5076c1c02ef3b2fa8fb0ce2405629aab3ff7fa3c)
    Reviewed-on: https://code.wireshark.org/review/29976
    

Actions performed:

    from  f15bf6a   Don't return TRUE with *err or *err_info set.
     add  cfb90bb   Clean up some things.


Summary of changes:
 wiretap/ascend.y     |  4 ++--
 wiretap/ascendtext.c | 59 ++++++++++++++++++++++++++++++----------------------
 2 files changed, 36 insertions(+), 27 deletions(-)