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

[Dmitry Timoshkov]

"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.

shlwapi: stub implementation for SHSetTimerQueueTimer (try 2)

[Nigel Liang (梁乃強)]

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...

-Nigel

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.

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.