Re: shlwapi: stub implementation for SHSetTimerQueueTimer (try 2)

[James Hawkins]

On 5/21/07, Nigel Liang (梁乃強)  wrote:
> Hi,
>
> I am not quite sure I understand what you mean? CreateTimerQueueTimer
> should return a Boolean indicating whether the operation has
> succeeded. Checking the return value in a ternary operation should
> return the correct handle if CreateTimerQueueTimer succeeds and NULL
> otherwise. Unless I have misunderstood the msdn documentation?
> http://msdn.microsoft.com/library/default.asp?url=/library/en-us/shellcc/pla...
> - return CreateTimerQueueTimer(&hNewTimer, hQueue, pfnCallback, pContext,
> -
>    dwDueTime, dwPeriod, dwFlags)?hNewTimer:NULL;

I'm pretty sure Dmitry did what I did.  That last little blurb at the
end (?hNewTimer:NULL;) is not very noticeable, and looks like the end
of the function arguments at first glance.  You really should get rid
of the ternary condition and split it out for code readability.

Also, please bottom-post on this list.
> On 5/21/07, Dmitry Timoshkov  wrote:
> > "Nigel Liang"  wrote:
> > > +HANDLE WINAPI SHSetTimerQueueTimer(HANDLE hQueue,
> > > -
> > >    WAITORTIMERCALLBACK pfnCallback, LPVOID pContext, DWORD dwDueTime,
> > > -
> > >    DWORD dwPeriod, LPCSTR lpszLibrary, DWORD dwFlags)
> > >
> > > +{
> >
> > ...
> > > - return CreateTimerQueueTimer(&hNewTimer, hQueue, pfnCallback, pContext,
> > > -
> > >    dwDueTime, dwPeriod, dwFlags)?hNewTimer:NULL;
> >
> > This can't work: CreateTimerQueueTimer and SHSetTimerQueueTimer
> > return different things.
> >
> > --
> > Dmitry.