>> @@ -976,13 +974,12 @@ static void handle_controller(_cmsg *cmsg) >> if (debugmode) >> printk(KERN_DEBUG "capidrv-%d: listenconf Info=0x%4x (%s) cipmask=0x%x\n", >> card->contrnr, cmsg->Info, capi_info2str(cmsg->Info), card->cipmask); >> - if (cmsg->Info) { >> + if (cmsg->Info) >> listen_change_state(card, EV_LISTEN_CONF_ERROR); >> - } else if (card->cipmask == 0) { >> + else if (card->cipmask == 0) >> listen_change_state(card, EV_LISTEN_CONF_EMPTY); >> - } else { >> + else > > Indented too much. How do you think about an alignment of this "else" with the corresponding if statement three lines above? Regards, Markus -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html