Hi,
>Is the second part of the patch (added inside the else clause)  correct?
Looks ok, perhap an assert or g_warning as it shouldn't happen?
Thanks for cleaning things up.
Best regards
Anders 

-----Original Message-----
From: [email protected] 
[mailto:[email protected]] On Behalf Of 
[email protected]
Sent: den 19 maj 2011 07:45
To: [email protected]
Subject: [Wireshark-commits] rev 37282: /trunk/gtk/ /trunk/gtk/: tcp_graph.c

http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=37282

User: jmayer
Date: 2011/05/18 10:44 PM

Log:
 Fix two warning in case of UIMANAGER:
 - Unused variable
 - 'graph_type' may be used uninitialized
 
 Is the second part of the patch (added inside the else clause)  correct?

Directory: /trunk/gtk/
  Changes    Path           Action
  +3 -1      tcp_graph.c    Modified

___________________________________________________________________________
Sent via:    Wireshark-commits mailing list <[email protected]>
Archives:    http://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
             mailto:[email protected]?subject=unsubscribe
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <[email protected]>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:[email protected]?subject=unsubscribe

Reply via email to