[PATCH] Changed assert to ASSERT

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/OpenBSC@lists.osmocom.org/.

Holger Hans Peter Freyther holger at freyther.de
Sat Mar 2 21:16:18 UTC 2013


On Fri, Mar 01, 2013 at 06:43:33PM +0000, Katerina Barone-Adesi wrote:

Dear Katerina,


> Rationale: zecke pointed out that the tests should unconditionally
> assert, regardless of debug settings.

> --- a/include/osmocom/core/utils.h
> +++ b/include/osmocom/core/utils.h

> +#define ASSERT(exp)    \

it is good thinking to share the code between multiple users (e.g. in
OpenBSC I have a bit of copy and paste for the assertion/testing macros)
that we could clean up.


One thing we have to be careful with is to not pollute the global 
namespace, e.g. with something as generic as 'ASSERT' we either need to
find another name or prefix it with OSMO_.

kind regards
	holger




More information about the OpenBSC mailing list