[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lwip-devel] [task #6792] Create ASSERTs for DEBUG compile
From: |
Jonathan Larmour |
Subject: |
[lwip-devel] [task #6792] Create ASSERTs for DEBUG compile |
Date: |
Wed, 23 May 2007 16:37:49 +0000 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.0.10) Gecko/20070223 Fedora/1.5.0.10-1.fc5 Firefox/1.5.0.10 |
Follow-up Comment #15, task #6792 (project lwip):
I disagree about NULL pointer checks in a release build. That would be a
programmer error. Unless you are referring to return values from calls to
e.g. mem_malloc
Otherwise, code should be debugged until it works. In most cases in embedded
stuff, if it doesn't work in the field, there's nothing you can do anyway.
What's the point of having an assert in a TV? Or an air conditioning unit?
(Yes those are two examples where customers of ours are using TCP/IP
stacks!). Or even a printer. In the field there's no point - use a watchdog
instead and just reset. Or if it's that important to you, then leave asserts
on in your release build. But for those who write programs that work, these
sorts of checks are a pointless waste of code.
_______________________________________________________
Reply to this item at:
<http://savannah.nongnu.org/task/?6792>
_______________________________________________
Message sent via/by Savannah
http://savannah.nongnu.org/
- [lwip-devel] [task #6792] Create ASSERTs for DEBUG compile, Simon Goldschmidt, 2007/05/14
- [lwip-devel] [task #6792] Create ASSERTs for DEBUG compile, Kieran Mansley, 2007/05/23
- [lwip-devel] [task #6792] Create ASSERTs for DEBUG compile, Jonathan Larmour, 2007/05/23
- [lwip-devel] [task #6792] Create ASSERTs for DEBUG compile, Simon Goldschmidt, 2007/05/23
- [lwip-devel] [task #6792] Create ASSERTs for DEBUG compile, Kieran Mansley, 2007/05/23
- [lwip-devel] [task #6792] Create ASSERTs for DEBUG compile, Simon Goldschmidt, 2007/05/23
- [lwip-devel] [task #6792] Create ASSERTs for DEBUG compile,
Jonathan Larmour <=
- [lwip-devel] [task #6792] Create ASSERTs for DEBUG compile, Simon Goldschmidt, 2007/05/23
- [lwip-devel] [task #6792] Create ASSERTs for DEBUG compile, Simon Goldschmidt, 2007/05/24
- [lwip-devel] [task #6792] Create ASSERTs for DEBUG compile, Simon Goldschmidt, 2007/05/24