cab_F_00: codename: "whee"

Gregory M. Turner gmturner007 at ameritech.net
Sun Jul 27 22:32:58 CDT 2003


On Sunday 27 July 2003 09:35 pm, Dimitrie O. Paun wrote:
> On July 27, 2003 04:46 pm, Gregory M. Turner wrote:
> > -              WARN("FDIIsCabinet failed.\n");
> > +              ERR("FDIIsCabinet failed.\n");
>
> Why these changes, they don't seem to make sense.
> The function handles the problem just fine, no need
> to spam the user with such messages, we already have
> too many IMO.

There is a "legitimate" way for this to occur, which makes it arguably 
excessive, but my reasoning was that 99% of the time this would instead be 
the result of some prior error which had occurred silently...  Since there 
was no memory leak FIXME in fdi.c we get a failure in msiexec or whatever and 
it gives some message like "File not found" but debugmsg +file reveals 
nothing because no file operation occurs...

I dunno I guess I thought maybe cabinet.dll would just be silent in this case 
and too easy to miss as a culprit...  maybe it's stupid.

-- 
gmt

#define SHOCK_TRACE(...) \
  do {\
    WINE_TRACE("\n\nXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX\n" \
               "X#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#X\n" \
               "XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX\n" \
               "@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@\n" \
               "XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX\n" \
               "X#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#X\n" \
               "XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX\n\n"); \
    WINE_TRACE(__VA_ARGS__); \
    WINE_TRACE("\n\nXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX\n" \
               "X#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#X\n" \
               "XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX\n" \
               "@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@\n" \
               "XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX\n" \
               "X#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#*#X\n" \
               "XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX\n\n"); \
  } while (0)




More information about the wine-devel mailing list