squalyl (./311) :
./308 ca aurait du sens si ils avaient écrit if(confirm(...) == something)

mais la ils font directement if(confirm(...)) donc la conversion implicite en bool est déja utilisée, ca ne fait aucun sens de vouloir l'éviter pour le retour de cette fonction.
ca ne ferait même pas de sens si la fonction appelante avait absolument besoin d'un bool. Puisque la conversion en bool se ferait au moment ou l'expression passée est évaluée, dans l'appelante.

Pas forcément...
Exemple de base : confirm() peut très bien se mettre à envoyer
0 = timeout
1 = ok
-1 = nok
-2 = interruption par un événement extérieur
2 = validation par un événement extérieur

S'il y a une évolution des données de sortie de confirm(), tu as juste à changer if (confirm()) en if (confirm() > 0), ce qui te permet d'éviter d'avoir à te repalucher tout ton code.