Re: [Nbd] [PATCH 1/1] server: remove non-reachable call to err()
- To: Tuomas R?s?nen <tuomasjjrasanen@...1261...>
- Cc: nbd-general@lists.sourceforge.net, Wouter Verhelst <w@...112...>, Tuomas R?s?nen <tuomasjjrasanen@...1258...>
- Subject: Re: [Nbd] [PATCH 1/1] server: remove non-reachable call to err()
- From: folkert <folkert@...421...>
- Date: Mon, 11 Mar 2013 21:07:55 +0100
- Message-id: <20130311200755.GK1911@...855...>
- In-reply-to: <20130310213810.GA22370@...1259...>
- References: <4872930b34a4b68d73810e4e31f9fa0d522e0aac.1362947875.git.tuomasjjrasanen@...1261...> <20130310211507.GA23754@...3...> <20130310213810.GA22370@...1259...>
> > Mmm. Checking for programmer error isn't necessarily a bad idea, though.
> > Yes, it's best done at compile time, but that isn't always possible.
> >
> > Perhaps change it into an assert() instead?
>
> Very well, assert() might be a good idea and I even thought it for a
> second or two, but then the pedantic side of me took
> over. Assert-statements do burn my eyes, you know. But that should be
> easily fixed by wearing PRAGMATISM-protective glasses more often while
> coding.
Didn't use asserts use cpu-time? E.g. aren't they also executed in
non-debug mode?
Folkert van Heusden
--
To MultiTail einai ena polymorfiko ergaleio gia ta logfiles kai tin
eksodo twn entolwn. Prosferei: filtrarisma, xrwmatismo, sygxwneysi,
diaforetikes provoles. http://www.vanheusden.com/multitail/
----------------------------------------------------------------------
Phone: +31-6-41278122, PGP-key: 1F28D8AE, www.vanheusden.com
Reply to: