[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