Dmitry Timoshkov writes:
> Alexandre Julliard wrote:
>
>> > The patch actually removes the casts, the only cast that remains is the
>> > cast
>> > on the caller's side for the vtable pointer (which is much cleaner IMO
>> > since
>> > every COM object implements IUnknown), but the implementatio
Alexandre Julliard wrote:
> > The patch actually removes the casts, the only cast that remains is the cast
> > on the caller's side for the vtable pointer (which is much cleaner IMO since
> > every COM object implements IUnknown), but the implementation itself is cast
> > free, and looks now as a
Dmitry Timoshkov writes:
> The patch actually removes the casts, the only cast that remains is the cast
> on the caller's side for the vtable pointer (which is much cleaner IMO since
> every COM object implements IUnknown), but the implementation itself is cast
> free, and looks now as an actual
Alexandre Julliard wrote:
> > This patch should be quite obvious in its intent: it changes vtable type
> > to IUnknown which helps to remove This* to IUnknown* casts in order to
> > avoid obvious problems with vtable pointer placement inside of object
> > implementation. Basically this is just a
Dmitry Timoshkov writes:
> This patch should be quite obvious in its intent: it changes vtable type
> to IUnknown which helps to remove This* to IUnknown* casts in order to
> avoid obvious problems with vtable pointer placement inside of object
> implementation. Basically this is just a clean up
This patch should be quite obvious in its intent: it changes vtable type
to IUnknown which helps to remove This* to IUnknown* casts in order to
avoid obvious problems with vtable pointer placement inside of object
implementation. Basically this is just a clean up patch.
Is there anything else to c