[U-Boot] [PATCH v2] zlib: fix code when DEBUG is defined

Wolfgang Denk wd at denx.de
Thu Sep 3 10:26:14 CEST 2009


Dear Giuseppe CONDORELLI,

In message <1251970209-9053-1-git-send-email-giuseppe.condorelli at st.com> you wrote:
> Removed stdio.h inclusion and moved trace macros to use printf avoiding to
> write debug informations to standard error.
> 
> Signed-off-by: Giuseppe Condorelli <giuseppe.condorelli at st.com>
> ---
>  lib_generic/zlib.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)

old:
	1 files changed, 29 insertions(+), 30 deletions(-)
new:
	1 files changed, 2 insertions(+), 2 deletions(-)

Isn't that great?

> @@ -2000,7 +2001,6 @@ void z_error (m)
>      char *m;
>  {
>  	fprintf(stderr, "%s\n", m);
> -	exit(1);
>  }
>  #endif

Sorry for not catching this in the first round.

What happens when we return here? Obviously this function is supposed
to never return. Should we call hang() here?

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
"Where shall I begin, please your Majesty?" he asked. "Begin  at  the
beginning,"  the  King said, gravely, "and go on till you come to the
end: then stop."    - Alice's Adventures in Wonderland, Lewis Carroll


More information about the U-Boot mailing list