[U-Boot] [PATCH] Add assert() for debug assertions

Simon Glass sjg at chromium.org
Thu Jun 23 23:35:34 CEST 2011


On Wed, Jun 22, 2011 at 3:08 PM, Mike Frysinger <vapier at gentoo.org> wrote:
> On Wednesday, June 22, 2011 17:56:49 Wolfgang Denk wrote:
>> Mike Frysinger wrote:
>> > the trouble with ifdef magic like this is that errors/warnings can be=20
>> > introduced when DEBUG isnt defined, and then only noticed when DEBUG
>> > is=20 defined.  so how about:
>> >
>> > #ifdef DEBUG
>> > # define _DEBUG 1
>> > #else
>> > # define _DEBUG 2
>>
>> 1 and 2?  You don't happen to mean 1 and 0 ?
>
> err, of course.  i had something semi-related on my mind as i banged that out.

Hi Mike & Wolfgang,

That sounds better thanks, I will send a new patch. It would be good
to have it compile the same code in and out of DEBUG mode.

Regards,
Simon

> -mike
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>
>


More information about the U-Boot mailing list