21 Jan
2005
21 Jan
'05
10:23 a.m.
Andreas Mohr wrote:
+ FIXME("%p %p %ld\n", hdc, p, count);
This is a step backwards IMHO. It doesn't tell squat: the user doesn't know that this is a stub ("FIXME" what??), and he doesn't know that these are supposed to be the function parameters either (it could be any other variable debug printout, too - it's indistinguishable).
Looking at logs tells the user nothing except that there's a problem. The only way to figure out why the message is there is to look at the code, and it should be pretty obvious if it's a stub or not. Mike