(C'è stato recentemente un thread nel forum in inglese in cui gente molto più preparata di me ha letteralmente "spaccato il capello in 4" (o 8, 16, ecc.))
if (boolean) è semanticamente corretto perché if si chiede se la condizione è vera o falsa. Il fatto che true e false vengano o meno risolte in 1 o 0 è irrilevante.
if (digitalRead(pin)) è semanticamente scorretto perché è come chiedersi if (int). Un interno è vero o falso ? Non ha senso. E' una domanda che evidentemente non ha senso. Mentre ha senso chiedersi se un intero vale 1 oppure no.
All'atto pratica la mia osservazione era legata solamente ad una maggiore leggiblità del codice (IMHO, ovviamente).
Mi scuso per aver sollevato questo vespaio, chiudiamola qui prima di scannarci per un nonnulla 