[flashrom] [PATCH] Standardize on using __func__ instead of __FUNCTION__
Uwe Hermann
uwe at hermann-uwe.de
Thu Sep 3 00:10:10 CEST 2009
On Wed, Sep 02, 2009 at 06:23:07PM +0200, Stefan Reinauer wrote:
> Uwe Hermann wrote:
> >See patch.
>
> For all debugging messages:
> Acked-by: Stefan Reinauer <stepan at coresystems.de>
Thanks, r711.
> But this:
>
> >Index: ft2232_spi.c
> >===================================================================
> >--- ft2232_spi.c (Revision 710)
> >+++ ft2232_spi.c (Arbeitskopie)
> >@@ -285,7 +285,7 @@
> > l = total_size - i;
> > if ((r = spi_nbyte_program(i, &buf[i], l))) {
> >- fprintf(stderr, "%s: write fail %d\n", __FUNCTION__, r);
> >+ fprintf(stderr, "%s: write fail %d\n", __func__, r);
> > return 1;
> > }
> >
> Error messages (ie. not _debugging_ messages) should not contain
> function names and numbers but descriptive error messages.
Agreed, this is for another patch, though.
Uwe.
--
http://www.hermann-uwe.de | http://www.holsham-traders.de
http://www.crazy-hacks.org | http://www.unmaintained-free-software.org
More information about the flashrom
mailing list