Re: [dev] [utmp][PATCH] Use same die() call in parent and child

From: Markus Teich <markus.teich_AT_stusta.mhn.de>
Date: Thu, 7 Aug 2014 18:13:17 +0200

Roberto E. Vargas Caballero wrote:
> I am used to fallthrough (maybe because I didn't put mark ever),
> but maybe is not the same for all the vibe-coders :?.
> This pull request is less about style that real benefit of the changes. I
> usually reuse all the possible paths of a hub to avoid code duplication.

Heyho Roberto,

usually the compiler should fix this code duplication, at most with -Os set.
So, before the pull request, the code is easier to read and the compiler takes care of
code deduplication. After the pull request you save some compiler optimization time
versus a little longer parsing time due to a few less lines of code which are
also more intuitive. I think we're better off without this one.

--Markus
Received on Thu Aug 07 2014 - 18:13:17 CEST