Wireshark · Wireshark-dev: Re: [Wireshark-dev] How does this not blow up?

[chuck c <bubbasnmp@xxxxxxxxx>]

It does "blow up" with 3.5.0rc0. Not sure why it works with 3.4.1.

On Thu, Dec 10, 2020 at 4:06 PM Jaap Keuter <jaap.keuter@xxxxxxxxx> wrote:

Hi,

In "ui/win32/file_dlg_win32.c"

static void
format_handle_wm_initdialog(HWND dlg_hwnd, print_args_t *args) {
    HWND cur_ctrl;
...
    switch (args->print_dissections) {
        case print_dissections_none:
        case print_dissections_collapsed:
            SendMessage(cur_ctrl, CB_SETCURSEL, 0, 0);
            break;
        case print_dissections_as_displayed:
            SendMessage(cur_ctrl, CB_SETCURSEL, 1, 0);
            break;
        case print_dissections_expanded:
            SendMessage(cur_ctrl, CB_SETCURSEL, 2, 0);
>> No break statement??
        default:
            g_assert_not_reached();
    }


___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
Archives:    https://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe