Wireshark-commits: [Wireshark-commits] master 096e523: Fix parsing of interface properties strings.
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Wed, 18 Mar 2015 23:58:59 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=096e5231ff41eb1b7c9c13fa1c6217fa78d73c6b
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

096e523 by Guy Harris (guy@xxxxxxxxxxxx):

    Fix parsing of interface properties strings.
    
    The first entry in the list does not have a leading comma, so a strstr()
    for ",{ifname}(" will not work.  Instead, use g_strsplit() with a comma
    for all of the preferences.
    
    Have common code for all the "uncomplicated" preferences.  Have a
    specialized handler for the one "complicated" preference.
    
    Change-Id: I2144a98ab0cb70db56eaaba88175d6e03885de2a
    Reviewed-on: https://code.wireshark.org/review/7741
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  9762c32   Complete the fix from g6b5c71a.
    adds  096e523   Fix parsing of interface properties strings.


Summary of changes:
 ui/capture_ui_utils.c               |  302 ++++++++++++++++++-----------------
 ui/capture_ui_utils.h               |    3 +-
 ui/gtk/capture_dlg.c                |    9 +-
 ui/gtk/prefs_capture.c              |    4 +-
 ui/iface_lists.c                    |    7 +-
 ui/qt/capture_interfaces_dialog.cpp |    4 +-
 6 files changed, 161 insertions(+), 168 deletions(-)