[U-Boot] [PATCH 4/8] net/tftp.c: cosmetic: do not use assignment in if condition

Wolfgang Denk wd at denx.de
Thu May 19 21:36:52 CEST 2011


Dear Luca Ceresoli,

In message <1305388203-22983-5-git-send-email-luca.ceresoli at comelit.it> you wrote:
> This removes the following checkpatch issue:
>  - ERROR: do not use assignment in if condition.
> 
> There is one such error left:
> 
>   ERROR: do not use assignment in if condition
>   #239: FILE: tftp.c:239:
>   +		if (!ProhibitMcast
>   +		 && (Bitmap = malloc(Mapsize))
>   +		 && eth_get_dev()->mcast) {
> 
> which would require an additional nested if to be fixed, resulting in longer
> and less readable code.
> 
> Signed-off-by: Luca Ceresoli <luca.ceresoli at comelit.it>
> Cc: Wolfgang Denk <wd at denx.de>
> ---
>  net/tftp.c |   18 ++++++++++++------
>  1 files changed, 12 insertions(+), 6 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"It's hard to believe that something which is neither seen  nor  felt
can do so much harm."
"That's true. But an idea can't be seen or felt. And that's what kept
the Troglytes in the mines all these centuries. A mistaken idea."
	-- Vanna and Kirk, "The Cloud Minders", stardate 5819.0


More information about the U-Boot mailing list