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.
[^] # Re: Hmm :/
Posté par Colin Leroy (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.