Wireshark-commits: [Wireshark-commits] master-2.4 ada54b8: ui: add missing break.

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Wed, 14 Nov 2018 09:31:50 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ada54b8ea9b234b608a3b94e62350054519b215f
Submitter: "Peter Wu <peter@xxxxxxxxxxxxx>"
Changed: branch: master-2.4
Repository: wireshark

Commits:

ada54b8 by Dario Lombardo (lomato@xxxxxxxxx):

    ui: add missing break.
    
    Error:
    ../ui/alert_box.c: In function ‘cfile_write_failure_alert_box’:
    ../ui/alert_box.c:359:13: error: this statement may fall through [-Werror=implicit-fallthrough=]
                 simple_error_message_box(
                 ^~~~~~~~~~~~~~~~~~~~~~~~~
                             "Frame %u%s has a network type that differs from the network type of earlier packets, which isn't supported in a \"%s\" file.",
                             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                             framenum, in_file_string,
                             ~~~~~~~~~~~~~~~~~~~~~~~~~
                             wtap_file_type_subtype_string(file_type_subtype));
                             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    ../ui/alert_box.c:364:9: note: here
             case WTAP_ERR_PACKET_TOO_LARGE:
             ^~~~
    
    Change-Id: I55464afff5625ae8c587470e417234560c7e606c
    Reviewed-on: https://code.wireshark.org/review/30623
    Reviewed-by: Dario Lombardo <lomato@xxxxxxxxx>
    Petri-Dish: Dario Lombardo <lomato@xxxxxxxxx>
    Tested-by: Dario Lombardo <lomato@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    (cherry picked from commit f3b82ffae752f59e22dee7506322e644ca2144d1)
    Reviewed-on: https://code.wireshark.org/review/30625
    

Actions performed:

    from  2713dcb   Catch attempts to write multiple encapsulation types if unsupported.
     add  ada54b8   ui: add missing break.


Summary of changes:
 ui/alert_box.c | 1 +
 1 file changed, 1 insertion(+)