Wireshark-commits: [Wireshark-commits] master fccfbf2: Stock icon (Qt): Missing break in switch (CI

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sun, 13 Sep 2015 17:52:06 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=fccfbf254e8b26a622420813a26b31edece343ca
Submitter: Alexis La Goutte (alexis.lagoutte@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

fccfbf2 by Alexis La Goutte (alexis.lagoutte@xxxxxxxxx):

    Stock icon (Qt): Missing break in switch (CID 1323928)
    
    Add a "// Fall through" comment to squelch a Coverity warning. Hopefully
    the regex described at
    
    http://security.coverity.com/blog/2013/Sep/gimme-a-break.html
    
    is case-insensitive.
    
    Change-Id: I10be63a77ed1e5e0c8000bad0c1b8d08db8b0951
    Reviewed-on: https://code.wireshark.org/review/10509
    Reviewed-by: Gerald Combs <gerald@xxxxxxxxxxxxx>
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    

Actions performed:

    from  2d466c0   androiddump: fix memory leaks reported by Coverity
    adds  fccfbf2   Stock icon (Qt): Missing break in switch (CID 1323928)


Summary of changes:
 ui/qt/stock_icon_tool_button.cpp |    1 +
 1 file changed, 1 insertion(+)