On Tuesday 03 July 2012 17:32:54 Marek Vasut wrote:
> > +int do_dfu(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
> > +{
> > +   char *str_env = NULL, *env_bkp = NULL;
> > +   static char *s = "dfu";
> > +   int ret = 0;
> > +
> > +   if (argc < 3)
> > +           return CMD_RET_USAGE;
> > +
> > +   str_env = getenv("dfu_alt_info");
> > +   if (str_env == NULL) {
> > +           printf("%s: \"dfu_alt_info\" env variable not defined!\n",
> > +                  __func__);
> 
> I was always curious if it's not possible to do something like
> 
> puts(__func__ "rest of string");
> 
> Maybe it'd help the overhead a bit? Certainly, it's beyond the scope of
> this patchset, I'm just curious :)

not anymore -- gcc disabled support for that sometime ago.  and it's good they 
did as it causes more bloat than good.  as soon as you do more than one 
statement, you get duplication.  the string table will have:
        "some_func: rest of string"
        "some_func: boo"
        "some_func: another message"
which takes up more space than:
        "some_func"
        "%s: rest of string"
        "%s: boo"
        "%s: another message"
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to