Wireshark-commits: [Wireshark-commits] master f0311ab: Rename update_interface_list() and remove lo
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, 21 Mar 2015 10:06:20 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=f0311abe72614e1ff9d653205f1eaf90f4350049
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

f0311ab by Guy Harris (guy@xxxxxxxxxxxx):

    Rename update_interface_list() and remove local interface code from it.
    
    It's only used to update the remote interface list, and never updates
    anything else (it sets iftype_id to CAPTURE_IFREMOTE and never changes
    it, so all tests for whether iftype_id is == or >= CAPTURE_IFREMOTE
    succeed); remove the code that would be executed if, somehow, iftype_id
    got stomped on, and rename it to update_remote_interface_list().
    
    Change-Id: I3bd80fb19bb14ef5bdf2c1a77a10f1f32bb62aba
    Reviewed-on: https://code.wireshark.org/review/7773
    Petri-Dish: Guy Harris <guy@xxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  2450bf9   Updated sercos discriptor CP0 for MDT0 and AT0
    adds  f0311ab   Rename update_interface_list() and remove local interface code from it.


Summary of changes:
 ui/gtk/capture_dlg.c |   40 +++++++++++++++-------------------------
 1 file changed, 15 insertions(+), 25 deletions(-)