[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 1/3] spd-conf revised to reflect the current state
From: |
Andrei Kholodnyi |
Subject: |
[PATCH 1/3] spd-conf revised to reflect the current state |
Date: |
Mon, 30 Aug 2010 13:51:27 +0200 |
> + assert(0)
>
> Do we really need this assert? IMO asserts are good for debugging, but
> not for the stable sw.
>
> This is only used to check internal consistency. If you compile
> with -O, it will not be in the final byte-compiled code. Which is
> exactly what assert is meant for.
What I'm actually saying here: there is no proper error handling in
place, but assert.
by removing assert you'll not add error handling.
Sorry for being not precise enough before.
- [PATCH 1/3] spd-conf revised to reflect the current state, (continued)
- [PATCH 1/3] spd-conf revised to reflect the current state, Hynek Hanke, 2010/08/30
- [PATCH 1/3] spd-conf revised to reflect the current state, Andrei Kholodnyi, 2010/08/30
- [PATCH 1/3] spd-conf revised to reflect the current state, Hynek Hanke, 2010/08/30
- [PATCH 1/3] spd-conf revised to reflect the current state, Andrei . Kholodnyi, 2010/08/30
- [PATCH 1/3] spd-conf revised to reflect the current state, Jan Buchal, 2010/08/30
- [PATCH 1/3] spd-conf revised to reflect the current state, Trevor Saunders, 2010/08/30
- [PATCH 1/3] spd-conf revised to reflect the current state, Jan Buchal, 2010/08/31
- [PATCH 1/3] spd-conf revised to reflect the current state, Trevor Saunders, 2010/08/31
- [PATCH 1/3] spd-conf revised to reflect the current state,
Andrei Kholodnyi <=