Wireshark-commits: [Wireshark-commits] master 4536271: Don't use a leading - in getopt_long() optio
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: Sat, 27 Dec 2014 02:47:08 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=4536271c0a8105b8a84a6b29200faf5477854b26
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

4536271 by Guy Harris (guy@xxxxxxxxxxxx):

    Don't use a leading - in getopt_long() option strings.
    
    The behavior of a leading - is platform-dependent.  It also means that
    non-option arguments are treated in a fashion that we're not handling,
    so capture filters given as non-option arguments at the end of the
    command line don't work.  (The Linux getopt() man page says that a
    leading - "is used by programs that were written to expect options and
    other argv-elements in any order and that care about the ordering of the
    two."  We are not such a program.)
    
    Change-Id: I5610cf90a8218d48f7516abacc367e0affa3b549
    Based-On-A-Change-From: Peter Hatina <phatina@xxxxxxxxxx>
    Reviewed-on: https://code.wireshark.org/review/6071
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  0a0ce2b   Check for optreset in CMake as well.
    adds  4536271   Don't use a leading - in getopt_long() option strings.


Summary of changes:
 tfshark.c |   13 +++++++++----
 tshark.c  |   13 +++++++++----
 2 files changed, 18 insertions(+), 8 deletions(-)