• [^] # Re: Hmm :/

    Posté par (site web personnel) . En réponse au journal Entretient du noyau Linux. Évalué à 4.


    1142 if (retval)
    1143 return ERR_PTR(retval);
    1144 return p;

    pourquoi ne pas mettre ca dans une fonction ?


    Pour que ce soit lisible.

    à mon avis le compilateur la inlinera très bien.

    (si on lui dit)


    gestion_erreur(code){
    switch(code){
    case ERROR_ALLOC_TRUC:
    do_stuff();
    case ...

    }
    }


    Et tu le mets où ton switch ? Dans une autre fonction ? avec la belle fonction inlinee précédemment ça sous-entend que chaque
    if(...)
    goto truc;

    se transforme en
    if (...) {
    gestion_erreur(code, toutes, mes, variables, locales, que, je, traite, dans, le cas, d'une erreur);
    return ma_belle_fonction_inlinee(retval, p);
    }

    Si tu le mets pas dans une autre fonction t'es obligé d'utiliser un goto de toutes façons.

    C'est tellement plus clair, moins dupliqué, plus lisible, tout ça.