le code geekesque du jour
Par gblanc le lundi, janvier 18 2010, 11:32 - informatique - Lien permanent
static void
to_be_or_not_to_be (void)
{
/*
* If all of the options that control our policy are disabled, then we
* have no point in living. Save the user some memory and exit.
*/
/* you used to say live and let live... */
gboolean live = FALSE;
size_t i;
/* ...but in this ever changing world in which we live in... */
for (i = 0; i < G_N_ELEMENTS (gvm_settings) && !live; i++) {
if (gvm_settings[i].type == TYPE_BOOL)
live = *((int *) gvm_settings[i].var);
}
/* makes you give it a cry... */
if (!live) {
dbg ("daemon exit: live and let die
");
exit (EXIT_SUCCESS);
}
}
Toi aussi, amuse-toi à compter les références dans ce morceau de code...
Commentaires
Geekesque ? C'est inutilement compliqué, et la variable "live" n'est pas utile.
Cf. http://pastebin.com/f66f88e46 pour le même code en simplifié.
De plus, il vous faut vraiment un " " à la fin de vos chaînes de debug ? Ça vous arrive de les construire en plusieurs appels successifs ?
(vous == eux, je n'avais pas vu d'où ça venait)
Tout d'abord... bonne année ! :)
Quand tu vois le code de ¤$@@^ de gnome (notamment avec des "goto out" pour exécuter deux lignes de free et un return... La variable "live" à la con paraît tout à coup un coup de génie), au moins, quand tu tombes sur ça, avec tes yeux qui saignent, tu esquisses un sourire... >_<"