Re: [PATCH 4/5] ntp.c code flow clenaups (from Ingo)

From: Roman Zippel
Date: Sat Mar 15 2008 - 13:24:40 EST


Hi,

On Sat, 15 Mar 2008, Ingo Oeser wrote:

> I don't understand, why the goto is required here.
>
> If you move these two cases in front of case "TIME_OK" and
> omit the break, you can remove the goto and the label "start_timer".
> Don't forget the /* fall through */ comment then.

Nice catch, I'll do this in the original patch.

bye, Roman
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/