Ticket #3431: mc-indroduce-wswitch-default-check.patch

File mc-indroduce-wswitch-default-check.patch, 24.2 KB (added by and, 9 years ago)

  • m4.include/mc-cflags.m4 

    introduce -Wswitch-default check because switch cases should almost always have a default case
1. to 'catch' an unexpected value
2. to handle 'default' actions, where the cases are for special behavior
3. to show someone reading your code that you've covered that case

please cross check for default stub case and add code comment at default case

Signed-off-by: Andreas Mohr <and@gmx.li>
    a b  
    7171dnl    MC_CHECK_ONE_CFLAG([-Wstrict-aliasing]) 
    7272    MC_CHECK_ONE_CFLAG([-Wstrict-prototypes]) 
    7373    MC_CHECK_ONE_CFLAG([-Wswitch]) 
     74    MC_CHECK_ONE_CFLAG([-Wswitch-default]) 
    7475    MC_CHECK_ONE_CFLAG([-Wtype-limits]) 
    7576    MC_CHECK_ONE_CFLAG([-Wundef]) 
    7677    MC_CHECK_ONE_CFLAG([-Wuninitialized]) 

  • lib/charsets.c

    a b  
    454454        case ESTR_FAILURE: 
    455455            ch = '.'; 
    456456            break; 
     457        default: 
     458            break; 
    457459        } 
    458460        str_close_conv (conv); 
    459461    } 
    … …  
    489491    case ESTR_FAILURE: 
    490492        ch = '.'; 
    491493        break; 
     494    default: 
     495        break; 
    492496    } 
    493497    return ch; 
    494498} 
    … …  
    523527    case ESTR_FAILURE: 
    524528        ch = '.'; 
    525529        break; 
     530    default: 
     531        break; 
    526532    } 
    527533    return ch; 
    528534} 
    … …  
    565571        case ESTR_FAILURE: 
    566572            ch = '.'; 
    567573            break; 
     574        default: 
     575            break; 
    568576        } 
    569577        str_close_conv (conv); 
    570578    } 

  • lib/filehighlight/get-color.c

    a b  
    197197        if (mc_fhl_is_special_door (fe)) 
    198198            my_color = TRUE; 
    199199        break; 
     200    default: 
     201        break; 
    200202    } 
    201203 
    202204    return (my_color) ? mc_filter->color_pair_index : -1; 
    … …  
    250252            if (ret > 0) 
    251253                return -ret; 
    252254            break; 
     255        default: 
     256            break; 
    253257        } 
    254258    } 
    255259    return NORMAL_COLOR; 

  • lib/lock.c

    a b  
    251251                g_free (msg); 
    252252                goto ret; 
    253253                break;          /* FIXME: unneeded? */ 
     254            default: 
     255                break; 
    254256            } 
    255257            g_free (msg); 
    256258        } 

  • lib/search/glob.c

    a b  
    146146            if (!escaped_mode) 
    147147                g_string_append_c (buff, '\\'); 
    148148            break; 
     149        default: 
     150            break; 
    149151        } 
    150152        g_string_append_c (buff, c); 
    151153        escaped_mode = FALSE; 

  • lib/serialize.c

    a b  
    330330            g_free (current_value); 
    331331            current_status = WAIT_GROUP; 
    332332            break; 
     333        default: 
     334            break; 
    333335        } 
    334336    } 
    335337    g_free (current_group); 

  • lib/strutil/strescape.c

    a b  
    8181            case '\0': 
    8282                g_string_append (ret, "\\0"); 
    8383                continue; 
     84            default: 
     85                break; 
    8486            } 
    8587        } 
    8688 
    … …  
    145147                case '0': 
    146148                    g_string_append_c (ret, '\0'); 
    147149                    continue; 
     150                default: 
     151                    break; 
    148152                } 
    149153            } 
    150154 

  • lib/strutil/strutil8bit.c

    a b  
    286286        case J_RIGHT: 
    287287            ident = width - length; 
    288288            break; 
     289        default: 
     290            break; 
    289291        } 
    290292 
    291293        if ((int) remain <= ident) 
    … …  
    330332        case J_RIGHT: 
    331333            ident = length - width; 
    332334            break; 
     335        default: 
     336            break; 
    333337        } 
    334338 
    335339        pos += ident; 

  • lib/strutil/strutilascii.c

    a b  
    230230        case J_RIGHT: 
    231231            ident = width - length; 
    232232            break; 
     233        default: 
     234            break; 
    233235        } 
    234236 
    235237        /* add space before text */ 
    … …  
    289291        case J_RIGHT: 
    290292            ident = length - width; 
    291293            break; 
     294        default: 
     295            break; 
    292296        } 
    293297 
    294298        /* copy substring text, substring start from ident and take width  

  • lib/strutil/strutilutf8.c

    a b  
    619619        case J_RIGHT: 
    620620            tool.ident = width - pre_form->width; 
    621621            break; 
     622        default: 
     623            break; 
    622624        } 
    623625 
    624626        utf8_tool_insert_space (&tool, tool.ident); 
    … …  
    647649        case J_RIGHT: 
    648650            tool.ident = width - pre_form->width; 
    649651            break; 
     652        default: 
     653            break; 
    650654        } 
    651655 
    652656        utf8_tool_skip_chars_to (&tool, 0); 

  • lib/strutil/xstrtol.c

    a b  
    163163                base = 1000; 
    164164                suffixes++; 
    165165                break; 
     166            default: 
     167                break; 
    166168            } 
    167169        } 
    168170 

  • lib/tty/color-ncurses.c

    a b  
    169169                                            COLOR_WHITE, COLOR_BLACK, 
    170170                                            COLOR_WHITE, COLOR_BLACK, A_UNDERLINE); 
    171171            break; 
     172        default: 
     173            break; 
    172174        } 
    173175    } 
    174176    else 

  • lib/tty/key.c

    a b  
    11231123        case KEY_KP_MULTIPLY: 
    11241124            c = '*'; 
    11251125            break; 
     1126        default: 
     1127            break; 
    11261128        } 
    11271129 
    11281130    return (mod | c); 

  • lib/tty/color-slang.c

    a b  
    165165            mc_tty_color_pair_init_special (mc_color_pair, 
    166166                                            "white", "black", "white", "black", SLTT_ULINE_MASK); 
    167167            break; 
     168        default: 
     169            break; 
    168170        } 
    169171    } 
    170172    else 

  • lib/util.c

    a b  
    296296            if (ret->len == 0) 
    297297                g_string_append_c (ret, '\\'); 
    298298            break; 
     299        default: 
     300            break; 
    299301        } 
    300302        g_string_append_c (ret, *s); 
    301303    } 
    … …  
    773775                            r = new_r + 1; 
    774776                            goto osc_out; 
    775777                        } 
     778                    default: 
     779                        break; 
    776780                    } 
    777781                } 
    778782              osc_out: 
    … …  
    854858            return COMPRESSION_BZIP; 
    855859        case 'h': 
    856860            return COMPRESSION_BZIP2; 
     861        default: 
     862            break; 
    857863        } 
    858864    } 
    859865 
    … …  
    901907        return "/ulzma" VFS_PATH_URL_DELIMITER; 
    902908    case COMPRESSION_XZ: 
    903909        return "/uxz" VFS_PATH_URL_DELIMITER; 
     910    default: 
     911        break; 
    904912    } 
    905913    /* Should never reach this place */ 
    906914    fprintf (stderr, "Fatal: decompress_extension called with an unknown argument\n"); 

  • lib/vfs/direntry.c

    a b  
    640640    case SEEK_END: 
    641641        offset += size; 
    642642        break; 
     643    default: 
     644        break; 
    643645    } 
    644646    if (offset < 0) 
    645647        FH->pos = 0; 
    … …  
    822824    case VFS_SETCTL_FLUSH: 
    823825        ((struct vfs_s_subclass *) path_element->class->data)->flush = 1; 
    824826        return 1; 
     827    default: 
     828        break; 
    825829    } 
    826830    return 0; 
    827831} 

  • lib/vfs/path.c

    a b  
    309309                case 'r': 
    310310                    path_element->port = 2; 
    311311                    break; 
     312                default: 
     313                    break; 
    312314                } 
    313315            } 
    314316    } 

  • lib/vfs/utilvfs.c

    a b  
    334334                case 'r': 
    335335                    path_element->port = 2; 
    336336                    break; 
     337                default: 
     338                    break; 
    337339                } 
    338340            } 
    339341    } 

  • src/vfs/smbfs/helpers/lib/kanji.c

    a b  
    673673                *out++ = jis_kso; 
    674674                shifted = _KJ_KANJI; 
    675675                break; 
     676            default: 
     677                break; 
    676678            } 
    677679            code = sjis2jis ((int) from[0] & 0xff, (int) from[1] & 0xff); 
    678680            *out++ = (code >> 8) & 0xff; 
    … …  
    689691                *out++ = jis_ksi; 
    690692                shifted = _KJ_ROMAN; 
    691693                break; 
     694            default: 
     695                break; 
    692696            } 
    693697            *out++ = *from++; 
    694698        } 
    … …  
    701705        *out++ = jis_ksi; 
    702706        shifted = _KJ_ROMAN; 
    703707        break; 
     708    default: 
     709        break; 
    704710    } 
    705711    *out = 0; 
    706712    if (overwrite) 
    … …  
    819825                *out++ = jis_kso; 
    820826                shifted = _KJ_KANJI; 
    821827                break; 
     828            default: 
     829                break; 
    822830            } 
    823831            code = sjis2jis ((int) from[0] & 0xff, (int) from[1] & 0xff); 
    824832            *out++ = (code >> 8) & 0xff; 
    … …  
    837845                *out++ = jis_so; 
    838846                shifted = _KJ_KANA; 
    839847                break; 
     848            default: 
     849                break; 
    840850            } 
    841851            *out++ = ((int) *from++) - 0x80; 
    842852        } 
    … …  
    854864                *out++ = jis_ksi; 
    855865                shifted = _KJ_ROMAN; 
    856866                break; 
     867            default: 
     868                break; 
    857869            } 
    858870            *out++ = *from++; 
    859871        } 
    … …  
    868880        *out++ = jis_si1; 
    869881        *out++ = jis_ksi; 
    870882        break; 
     883    default: 
     884        break; 
    871885    } 
    872886    *out = 0; 
    873887    if (overwrite) 
    … …  
    980994                *out++ = jis_so2; 
    981995                shifted = _KJ_KANJI; 
    982996                break; 
     997            default: 
     998                break; 
    983999            } 
    9841000            code = sjis2jis ((int) from[0] & 0xff, (int) from[1] & 0xff); 
    9851001            *out++ = (code >> 8) & 0xff; 
    … …  
    9971013                *out++ = junet_kana2; 
    9981014                shifted = _KJ_KANA; 
    9991015                break; 
     1016            default: 
     1017                break; 
    10001018            } 
    10011019            *out++ = ((int) *from++) - 0x80; 
    10021020        } 
    … …  
    10111029                *out++ = jis_si2; 
    10121030                shifted = _KJ_ROMAN; 
    10131031                break; 
     1032            default: 
     1033                break; 
    10141034            } 
    10151035            *out++ = *from++; 
    10161036        } 
    … …  
    10231043        *out++ = jis_si1; 
    10241044        *out++ = jis_si2; 
    10251045        break; 
     1046    default: 
     1047        break; 
    10261048    } 
    10271049    *out = 0; 
    10281050    if (overwrite) 

  • src/vfs/smbfs/helpers/lib/util_sock.c

    a b  
    174174                                  socket_options[i].option, (char *) &on, sizeof (int)); 
    175175            } 
    176176            break; 
     177        default: 
     178            break; 
    177179        } 
    178180 
    179181        if (ret != 0) 

  • src/vfs/smbfs/helpers/libsmb/clientgen.c

    a b  
    21422142            return (ret); 
    21432143        } 
    21442144        return (SVAL (p, 0)); 
     2145    default: 
     2146        break; 
    21452147    } 
    21462148 
    21472149    DEBUG (1, ("Unknown long filename format %d\n", level)); 
    … …  
    22772279                pstrcpy (mask, p + ff_lastname + 1); 
    22782280                ff_resume_key = 0; 
    22792281                break; 
     2282            default: 
     2283                break; 
    22802284            } 
    22812285        } 
    22822286        else 
    … …  
    26642668            return ENOTDIR; 
    26652669        case NT_STATUS_OBJECT_NAME_COLLISION: 
    26662670            return EEXIST; 
     2671        default: 
     2672            break; 
    26672673        } 
    26682674 
    26692675        /* for all other cases - a default code */ 
    … …  
    26982704            return EBUSY; 
    26992705        case ERRlock: 
    27002706            return EBUSY; 
     2707        default: 
     2708            break; 
    27012709        } 
    27022710    } 
    27032711    if (rcls == ERRSRV) 
    … …  
    27142722            return ENODEV; 
    27152723        case ERRinvnetname: 
    27162724            return ENODEV; 
     2725        default: 
     2726            break; 
    27172727        } 
    27182728    } 
    27192729    /* for other cases */ 

  • src/vfs/smbfs/helpers/param/loadparm.c

    a b  
    20802080        break; 
    20812081    case P_SEP: 
    20822082        break; 
     2083    default: 
     2084        break; 
    20832085    } 
    20842086 
    20852087    return (True); 
    … …  
    21422144        } 
    21432145    case P_SEP: 
    21442146        break; 
     2147    default: 
     2148        break; 
    21452149    } 
    21462150    return (False); 
    21472151} 
    … …  
    23812385            break; 
    23822386        case P_SEP: 
    23832387            break; 
     2388        default: 
     2389            break; 
    23842390        } 
    23852391    } 
    23862392    defaults_saved = True; 

  • src/vfs/smbfs/smbfs.c

    a b  
    572572    case STYPE_IPC: 
    573573        typestr = "IPC"; 
    574574        break; 
     575    default: 
     576        break; 
    575577    } 
    576578    DEBUG (3, ("\t%-15.15s%-10.10s%s\n", name, typestr, comment)); 
    577579} 
    … …  
    18601862        } 
    18611863        info->nread = size + offset; 
    18621864        break; 
     1865    default: 
     1866        break; 
    18631867    } 
    18641868 
    18651869    return info->nread; 
    … …  
    20302034    case VFS_SETCTL_LOGFILE: 
    20312035        smbfs_set_debugf ((const char *) arg); 
    20322036        break; 
     2037    default: 
     2038        break; 
    20332039    } 
    20342040    return 0; 
    20352041} 

  • lib/widget/dialog.c

    a b  
    242242        case SELECT_PREV: 
    243243            h->current = dlg_widget_prev (h, h->current); 
    244244            break; 
     245        default: 
     246            break; 
    245247        } 
    246248    } 
    247249    while (h->current != w /* && (WIDGET (h->current->data)->options & W_DISABLED) == 0 */ ); 

  • lib/widget/input_complete.c

    a b  
    659659            if (!found) 
    660660                MC_PTR_FREE (cur_word); 
    661661        } 
     662    default: 
     663        break; 
    662664    } 
    663665 
    664666    if (found == NULL) 
    … …  
    10881090                    /* fallthrough */ 
    10891091                case -2: 
    10901092                    return MSG_HANDLED; 
     1093                default: 
     1094                    break; 
    10911095                } 
    10921096 
    10931097                for (i = 0, e = listbox_get_first_link (LISTBOX (h->current->data)); 

  • lib/widget/menu.c

    a b  
    484484    case XCTRL ('f'): 
    485485        menubar_right (menubar); 
    486486        return 1; 
     487    default: 
     488        break; 
    487489    } 
    488490 
    489491    if (!menubar->is_dropped) 
    … …  
    554556        case XCTRL ('p'): 
    555557            menubar_up (menubar); 
    556558            break; 
     559        default: 
     560            break; 
    557561        } 
    558562    } 
    559563 

  • lib/widget/radio.c

    a b  
    105105                r->pos++; 
    106106                return MSG_HANDLED; 
    107107            } 
     108        default: 
     109            break; 
    108110        } 
    109111        return MSG_NOT_HANDLED; 
    110112 

  • src/background.c

    a b  
    316316            case 4: 
    317317                result = routine.have_ctx4 (Background, data[0], data[1], data[2], data[3]); 
    318318                break; 
     319            default: 
     320                break; 
    319321            } 
    320322        else 
    321323            switch (argc) 
    … …  
    336338                result = 
    337339                    routine.non_have_ctx4 (ctx, Background, data[0], data[1], data[2], data[3]); 
    338340                break; 
     341            default: 
     342                break; 
    339343            } 
    340344 
    341345        /* Send the result code and the value for shared variables */ 

  • src/consaver/cons.saver.c

    a b  
    267267        case CONSOLE_CONTENTS: 
    268268            send_contents (buffer + 4, winsz.ws_col, winsz.ws_row); 
    269269            break; 
     270        default: 
     271            break; 
    270272        } 
    271273 
    272274        if (write (1, &console_flag, 1) != 1) 

  • src/diffviewer/ydiff.c

    a b  
    20042004        case CHG_CH: 
    20052005            res = DIFF_CHG; 
    20062006            break; 
     2007        default: 
     2008            break; 
    20072009        } 
    20082010        while (pos > 0 && ((DIFFLN *) & g_array_index (a0, DIFFLN, pos))->ch != EQU_CH) 
    20092011            pos--; 
    … …  
    22392241        case DIFF_CHG: 
    22402242            dview_replace_hunk (dview, merge_file, from1, to1, from2, to2, merge_direction); 
    22412243            break; 
     2244        default: 
     2245            break; 
    22422246        } 
    22432247        fflush (merge_file); 
    22442248        fclose (merge_file); 

  • src/editor/edit.c

    a b  
    11681168        case COLUMN_OFF: 
    11691169            edit->column_highlight = 0; 
    11701170            break; 
     1171        default: 
     1172            break; 
    11711173        } 
    11721174        if (ac >= 256 && ac < 512) 
    11731175            edit_insert_ahead (edit, ac - 256); 
    … …  
    12511253        case COLUMN_OFF: 
    12521254            edit->column_highlight = 0; 
    12531255            break; 
     1256        default: 
     1257            break; 
    12541258        } 
    12551259        if (ac >= 256 && ac < 512) 
    12561260            edit_insert_ahead (edit, ac - 256); 
    … …  
    19131917                break; 
    19141918            case LB_ASIS:      /* default without changes */ 
    19151919                break; 
     1920            default: 
     1921                break; 
    19161922            } 
    19171923        } 
    19181924    } 
    … …  
    34043410            edit->column_highlight = 0; 
    34053411            edit_mark_cmd (edit, TRUE); 
    34063412        } 
     3413    default: 
     3414        break; 
    34073415    } 
    34083416 
    34093417    switch (command) 
    … …  
    34293437    case CK_MarkLeft: 
    34303438    case CK_MarkRight: 
    34313439        edit->force |= REDRAW_CHAR_ONLY; 
     3440    default: 
     3441        break; 
    34323442    } 
    34333443 
    34343444    /* basic cursor key commands */ 
    … …  
    39613971        case CK_DeleteToEnd: 
    39623972            format_paragraph (edit, FALSE); 
    39633973            edit->force |= REDRAW_PAGE; 
     3974        default: 
     3975            break; 
    39643976        } 
    39653977    } 
    39663978} 

  • src/editor/editcmd.c

    a b  
    28792879    case 2:                    /* Cancel quit */ 
    28802880    case -1:                   /* Esc */ 
    28812881        return FALSE; 
     2882    default: 
     2883        break; 
    28822884    } 
    28832885 
    28842886    return TRUE; 

  • src/editor/editwidget.c

    a b  
    14591459            break; 
    14601460        } 
    14611461        break; 
     1462    default: 
     1463        break; 
    14621464    } 
    14631465 
    14641466    return ret; 

  • src/editor/etags.c

    a b  
    153153            *short_name = shortdef; 
    154154            *line = atol (linedef); 
    155155            return TRUE; 
     156        default: 
     157            break; 
    156158        } 
    157159        buf++; 
    158160        c = *buf; 
    … …  
    250252                } 
    251253            } 
    252254            break; 
     255        default: 
     256            break; 
    253257        } 
    254258    } 
    255259 

  • src/filemanager/achown.c

    a b  
    303303                h->ret_value = parm; 
    304304                dlg_stop (h); 
    305305            } 
     306        default: 
     307            break; 
    306308        } 
    307309 
    308310    default: 
    … …  
    615617                    dlg_one_down (h); 
    616618            } 
    617619            break; 
     620        default: 
     621            break; 
    618622        } 
    619623        return MSG_NOT_HANDLED; 
    620624 
    … …  
    888892 
    889893        case B_SKIP: 
    890894            break; 
     895        default: 
     896            break; 
    891897        } 
    892898 
    893899        if (current_panel->marked && result != B_CANCEL) 

  • src/filemanager/chmod.c

    a b  
    541541 
    542542            apply_mask (&sf_stat); 
    543543            break; 
     544        default: 
     545            break; 
    544546        } 
    545547 
    546548        if (current_panel->marked != 0 && result != B_CANCEL) 

  • src/filemanager/chown.c

    a b  
    436436                    apply_chowns (new_user, new_group); 
    437437                break; 
    438438            } 
     439        default: 
     440            break; 
    439441        }                       /* switch */ 
    440442 
    441443        if (current_panel->marked && ch_dlg->ret_value != B_CANCEL) 

  • src/filemanager/command.c

    a b  
    181181                state = copy_sym; 
    182182                break; 
    183183            } 
     184        default: 
     185            break; 
    184186        } 
    185187    } 
    186188 

  • src/filemanager/hotlist.c

    a b  
    15381538        case HL_TYPE_DOTDOT: 
    15391539            /* do nothing */ 
    15401540            break; 
     1541        default: 
     1542            break; 
    15411543        } 
    15421544} 
    15431545 

  • src/filemanager/layout.c

    a b  
    10221022 
    10231023        mcview_load ((struct mcview_struct *) new_widget, 0, file_name, 0); 
    10241024        break; 
     1025    default: 
     1026        break; 
    10251027    } 
    10261028 
    10271029    if (type != view_listing) 

  • src/filemanager/panel.c

    a b  
    33763376        /* reset state of marks flag */ 
    33773377        state_mark = -1; 
    33783378        break; 
     3379    default: 
     3380        break; 
    33793381    } 
    33803382    switch (command) 
    33813383    { 
    … …  
    35253527    case CK_SortByMTime: 
    35263528        panel_set_sort_type_by_id (panel, "mtime"); 
    35273529        break; 
     3530    default: 
     3531        break; 
    35283532    } 
    35293533    return res; 
    35303534} 

  • src/filemanager/panelize.c

    a b  
    549549            return; 
    550550        } 
    551551        break; 
     552    default: 
     553        break; 
    552554    } 
    553555 
    554556    panelize_done (); 

  • src/filemanager/treestore.c

    a b  
    192192        case '\\': 
    193193            *q = '\\'; 
    194194            break; 
     195        default: 
     196            break; 
    195197        } 
    196198    } 
    197199    *q = *p; 

  • src/filemanager/usermenu.c

    a b  
    905905            result = g_string_free (block, FALSE); 
    906906            goto ret; 
    907907        }                       /* sub case block */ 
     908    default: 
     909        break; 
    908910    }                           /* switch */ 
    909911    result = g_strdup ("% "); 
    910912    result[1] = c; 

  • src/learn.c

    a b  
    218218    case 'k': 
    219219        dlg_one_up (learn_dlg); 
    220220        return TRUE; 
     221    default: 
     222        break; 
    221223    } 
    222224 
    223225    /* Prevent from disappearing if a non-defined sequence is pressed 

  • src/subshell.c

    a b  
    335335    case FISH: 
    336336        execl (mc_global.tty.shell, "fish", (char *) NULL); 
    337337        break; 
     338    default: 
     339        break; 
    338340    } 
    339341 
    340342    /* If we get this far, everything failed miserably */ 
    … …  
    770772        mc_global.tty.use_subshell = FALSE; 
    771773        mc_global.midnight_shutdown = TRUE; 
    772774        return; 
     775    default: 
     776        break; 
    773777    } 
    774778 
    775779    /* Take the current (hopefully pristine) tty mode and make */ 
    … …  
    897901                    "function fish_prompt; echo $PWD>&%d; fish_prompt_mc; kill -STOP %%self; end\n", 
    898902                    subshell_pipe[WRITE]); 
    899903        break; 
    900  
     904    default: 
     905        break; 
    901906    } 
    902907    write_all (mc_global.tty.subshell_pty, precmd, strlen (precmd)); 
    903908 

  • src/vfs/cpio/cpio.c

    a b  
    751751            continue; 
    752752        case STATUS_TRAIL: 
    753753            break; 
     754        default: 
     755            break; 
    754756        } 
    755757        break; 
    756758    } 

  • src/vfs/fish/fish.c

    a b  
    829829                ST.st_rdev = makedev (maj, min); 
    830830#endif 
    831831            } 
     832        default: 
     833            break; 
    832834        } 
    833835    } 
    834836 

  • src/vfs/ftpfs/ftpfs.c

    a b  
    410410            if (string_buf != NULL) 
    411411                g_strlcpy (string_buf, answer, string_len); 
    412412            return code / 100; 
     413        default: 
     414            break; 
    413415        } 
    414416    } 
    415417} 
    … …  
    11451147    case AF_INET6: 
    11461148        ((struct sockaddr_in6 *) sa)->sin6_port = port; 
    11471149        break; 
     1150    default: 
     1151        break; 
    11481152    } 
    11491153 
    11501154    return (connect (my_socket, (struct sockaddr *) sa, *salen) < 0) ? 0 : 1; 

  • src/vfs/sfs/sfs.c

    a b  
    188188            case '%': 
    189189                COPY_CHAR; 
    190190                continue; 
     191            default: 
     192                break; 
    191193            } 
    192194            if (ptr != NULL) 
    193195            { 

  • src/vfs/sftpfs/file.c

    a b  
    399399        } 
    400400        file_handler->pos = file_handler->ino->st.st_size - offset; 
    401401        break; 
     402    default: 
     403        break; 
    402404    } 
    403405 
    404406    file_handler_data = (sftpfs_file_handler_data_t *) file_handler->data; 

  • src/vfs/tar/tar.c

    a b  
    423423            st->st_rdev = 
    424424                (tar_from_oct (8, header->header.devmajor) << 8) | 
    425425                tar_from_oct (8, header->header.devminor); 
     426        default: 
     427            break; 
    426428        } 
    427429    default: 
    428430        st->st_uid = tar_from_oct (8, header->header.uid); 
    … …  
    800802 
    801803            case STATUS_EOF: 
    802804                return 0; 
     805            default: 
     806                break; 
    803807            } 
    804808 
    805809            /* Record of zeroes */ 
    … …  
    808812            /* exit from loop */ 
    809813        case STATUS_EOF:       /* End of archive */ 
    810814            break; 
     815        default: 
     816            break; 
    811817        } 
    812818        break; 
    813819    } 

  • src/viewer/coord_cache.c

    a b  
    417417        case NROFF_BACKSPACE: 
    418418            nroff_state = NROFF_CONTINUATION; 
    419419            break; 
     420        default: 
     421            break; 
    420422        } 
    421423 
    422424        /* Cache entries must guarantee that for each i < j, 

  • src/viewer/datasource.c

    a b  
    177177        break; 
    178178    case DS_NONE: 
    179179        break; 
     180    default: 
     181        break; 
    180182    } 
    181183 
    182184    if (str == NULL) 

  • src/viewer/inlines.h

    a b  
    109109        return mcview_get_byte_string (view, offset, retval); 
    110110    case DS_NONE: 
    111111        return mcview_get_byte_none (view, offset, retval); 
     112    default: 
     113        break; 
    112114    } 
    113115#ifdef HAVE_ASSERT_H 
    114116    assert (!"Unknown datasource type");