[PATCH try3] atl110: Added new DLL.

Jacek Caban jacek at codeweavers.com
Tue Sep 3 06:42:00 CDT 2013


On 09/03/13 13:28, Qian Hong wrote:
> On Tue, Sep 3, 2013 at 7:27 PM, Qian Hong <fracting at gmail.com> wrote:
>> Hi Jacek, we already have a debug channel atl100 for atl100.dll, but
>> we currently use atl for both atl.dll and atl80.dll, do you think it
>> is better to use atl for all, or one debug channel per each dll?
>
> Oh, I just found dlls/atl100/atl_ax.c  and dlls/atl100/atl.c used
> different debug channel, is that expected?

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.

Thanks,
Jacek



More information about the wine-devel mailing list