if ( !wxTimerImpl::Start(milliseconds, oneShot) )
return false;
- m_id = ::SetTimer(
- wxTimerHiddenWindowModule::GetHWND(), // window to send the messages to
- GetId(), // timer ID
- (UINT)m_milli, // delay
- NULL // timer proc. Not used since we pass hwnd
- );
+ // SetTimer() doesn't accept 0 timer id so use something else if the timer
+ // id at wx level is 0: as -1 (wxID_ANY) can't be used, we can safely
+ // replace 0 with it at MSW level
+ UINT idTimer = GetId();
+ if ( !idTimer )
+ idTimer = (UINT)-1;
+
+ // SetTimer() normally returns just idTimer but this might change in the
+ // future so use its return value to be safe
+ m_id = ::SetTimer
+ (
+ wxTimerHiddenWindowModule::GetHWND(), // window for WM_TIMER
+ idTimer, // timer ID to create
+ (UINT)m_milli, // delay
+ NULL // timer proc (unused)
+ );
if ( !m_id )
{
void wxMSWTimerImpl::Stop()
{
- wxASSERT_MSG( m_id, _T("should be running") );
+ wxASSERT_MSG( m_id, wxT("should be running") );
::KillTimer(wxTimerHiddenWindowModule::GetHWND(), m_id);
void wxProcessTimer(wxMSWTimerImpl& timer)
{
- wxASSERT_MSG( timer.IsRunning(), _T("bogus timer id") );
+ wxASSERT_MSG( timer.IsRunning(), wxT("bogus timer id") );
if ( timer.IsOneShot() )
timer.Stop();
{
if ( !::DestroyWindow(ms_hwnd) )
{
- wxLogLastError(_T("DestroyWindow(wxTimerHiddenWindow)"));
+ wxLogLastError(wxT("DestroyWindow(wxTimerHiddenWindow)"));
}
ms_hwnd = NULL;
{
if ( !::UnregisterClass(ms_className, wxGetInstance()) )
{
- wxLogLastError(_T("UnregisterClass(\"wxTimerHiddenWindow\")"));
+ wxLogLastError(wxT("UnregisterClass(\"wxTimerHiddenWindow\")"));
}
ms_className = NULL;
/* static */
HWND wxTimerHiddenWindowModule::GetHWND()
{
- static const wxChar *HIDDEN_WINDOW_CLASS = _T("wxTimerHiddenWindow");
+ static const wxChar *HIDDEN_WINDOW_CLASS = wxT("wxTimerHiddenWindow");
if ( !ms_hwnd )
{
ms_hwnd = wxCreateHiddenWindow(&ms_className, HIDDEN_WINDOW_CLASS,