Re: [PATCH 3/4] oleacc: Add ObjectFromLresult implementation

[Piotr Caban]

On 04/25/14 14:02, Dmitry Timoshkov wrote:
> Piotr Caban  wrote:
> > On 04/25/14 13:34, Qian Hong wrote:
> > > - atom = result;
> > > - if(atom != result)
> > > -
> > >    return E_FAIL;
> > >
> > > I can't understand these few lines, are they typo or did I miss anything?
> >
> > result and atom has different size. This code checks if LRESULT
> > describes atom or error.
>
> GlobalGetAtomNameW should handle this just fine, 'atom' is a misleading
> intermediate variable IMO.

No, it's not handling it. It can happen that cast from an error code is 
a valid atom.