From mboxrd@z Thu Jan  1 00:00:00 1970
From: Helmut Raiger <helmut.raiger@hale.at>
Date: Thu, 14 Jul 2011 11:14:07 +0200
Subject: [U-Boot] [PATCH 1/2] net/eth.c: make eth_get_dev_by_name(NULL)
 safe
In-Reply-To: <m2wrfmv1wq.fsf@ohwell.denx.de>
References: <4E118AD9.9000200@hale.at>	<CAJaTeTpXocZhUQOv=8O58wbxLozsZGYScctcjDnpKm46q4BwRw@mail.gmail.com>	<4E1AC82D.3020205@hale.at>
	<201107120237.33837.vapier@gentoo.org>	<m2d3hfg8fo.fsf@ohwell.denx.de>
	<4E1D3C05.9020808@hale.at> <m2wrfmv1wq.fsf@ohwell.denx.de>
Message-ID: <4E1EB35F.1060400@hale.at>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 07/13/2011 01:46 PM, Detlev Zundel wrote:
>
>> The NDEBUG approach however, as Mike suggested,  was what I was
>> looking for in the first place.
> Great!
>    Detlev
>

Again, not so great. U-boot uses all kinds of assert(), BUG_ON(), 
ASSERT() all over the place.
This probably needs a project wide effort, which is why I simply threw 
in my NULL pointer check.

Helmut


--
Scanned by MailScanner.