Wireshark · Wireshark-commits: [Wireshark-commits] master a972e8f: Try to fix a used-but-not-set warning.

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=a972e8f2b3615c07b9dad28611c00c158a9b103c
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

a972e8f by Guy Harris (guy@xxxxxxxxxxxx):

    Try to fix a used-but-not-set warning.
    
    I'm guessing that GCC is complaining because "device" isn't guaranteed
    to be set (if there are no interfaces, the body of the loop will never
    be executed), so do with the list of link-layer header types what's done
    for the buffer size and snapshot length.
    
    But should the column-setting code be executed if the interface isn't
    found?
    
    Change-Id: I8c365a482b8af44cdd1c1a7f41ca611141387c86
    Reviewed-on: https://code.wireshark.org/review/2180
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  c246513   Don't check "&(device.if_info)" to see if it's null.
    adds  a972e8f   Try to fix a used-but-not-set warning.


Summary of changes:
 ui/qt/capture_interfaces_dialog.cpp |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)