On Tue, Sep 3, 2013 at 7:42 PM, Jacek Caban jacek@codeweavers.com wrote:
Not really, good catch. We should make them consistent. Honestly, I'm not sure which one is better. Both have their problems. Some functions are forwarded, others are not, so having one debug channel would be guarantee that we don't miss some calls while debugging a bug. However, some functions have the same names and are not forwarded, so one debug channel would be ambiguous.
I'm open for opinions.
How about something like this:
atl80.c: -BOOL WINAPI AtlAxWinInit(void) +BOOL WINAPI ATL80_AtlAxWinInit(void)
atl80.spec: -42 stdcall AtlAxWinInit() +42 stdcall AtlAxWinInit() ATL80_AtlAxWinInit
So we can always use one debug channel for all atlXX dlls, at the same time different exported function with the same name will generate different trace log.