win32 - timeSetEvent(): what is wrong with these function?

**rcgldr** · 04-30-2016

Seems like you only get one timeSetEvent with your setup. Perhaps the call back function could set an array of events, each of which in turn would call back TimerProc() .

**joaquim** · 04-30-2016

what you advice?
heres my entire Timer class:

Code:

class Timer
{

private:
    UINT_PTR timerid=0;
    UINT m_uResolution=0;
    unsigned int intInterval=0;

    static void CALLBACK _TimerProc(UINT wTimerID, UINT msg, DWORD_PTR dwUser, DWORD dw1, DWORD dw2)
    {
        Timer* obj=reinterpret_cast<Timer*>(dwUser);
        obj->timerprocedure();
    }

public:

    std::function<void()> timerprocedure=NULL;
    Timer(std::function<void()> tmrprocedure=NULL)
    {
        timerprocedure=tmrprocedure;
    }

    void Stop()
    {
        if(timerid!=0)
        {
            timeKillEvent(timerid);
            timeEndPeriod (m_uResolution);
        }
    }

    property <unsigned int> Interval
    {
        Get(unsigned int)
        {
            return intInterval;
        },
        Set(unsigned int uintInterval)
        {
            intInterval = uintInterval;
        }
    };

    void Start()
    {
        Stop();
        TIMECAPS tc;
        timeGetDevCaps(&tc, sizeof(TIMECAPS));
        m_uResolution = min(max(tc.wPeriodMin, 0), tc.wPeriodMax);
        timeBeginPeriod(m_uResolution);
        timerid = timeSetEvent(intInterval, m_uResolution, Timer::_TimerProc, reinterpret_cast<DWORD_PTR>(this),TIME_PERIODIC | TIME_KILL_SYNCHRONOUS);

        if (timerid==0)
            DebugText("error\t" + to_string(GetLastError()));
    }

    ~Timer()
    {
        Stop();
    }
};

timerprocedure it's a lambda. i don't know if i can use it on array way.

**rcgldr** · 05-02-2016

I meant an array or set of events. Use timeSetEvent to call a helper function. That function in turn will set up to 7 events, each which in turn can call a timer function,

**joaquim** · 05-02-2016

i'm sorry. but don't make sence to me

thinking on that: what is for the 'this'?

**rcgldr** · 05-02-2016

It would help if you could explain how the 7 instances of the timer class are being used.

**rcgldr** · 05-03-2016

I'm able to create two timers with Visual Studio / Win XP, so there may be an issue with your setup. This example is based on old code.

Code:

#include <windows.h>
#include <stdio.h>

#pragma comment(lib, "winmm.lib")       /* include winmm.lib */

/*----------------------------------------------------------------------*/
/*      data                                                            */
/*----------------------------------------------------------------------*/
static TIMECAPS sTimeCaps;              /* min and max timer values */
static MMRESULT mmTimerId0;             /* timer id */
static MMRESULT mmTimerId1;             /* timer id */
static volatile DWORD dwTicks0;         /* tick counter */
static volatile DWORD dwTicks1;         /* tick counter */

/*----------------------------------------------------------------------*/
/*      code                                                            */
/*----------------------------------------------------------------------*/
void CALLBACK TimerFunction0(UINT, UINT, DWORD, DWORD, DWORD);
void CALLBACK TimerFunction1(UINT, UINT, DWORD, DWORD, DWORD);

/*----------------------------------------------------------------------*/
/*      main                                                            */
/*----------------------------------------------------------------------*/
int main(int argc, char **argv)
{
/*                                      ** get min and max period */
    timeGetDevCaps(&sTimeCaps, sizeof(sTimeCaps));
    if(sTimeCaps.wPeriodMin != 1){
        printf("1 ms resolution not allowed\n");
        goto exit1;}

    timeBeginPeriod(1);                 /* set period to 1ms */
    Sleep(128);                         /* wait for it to stabilize */

/*                                      ** create two 4ms timers */
    mmTimerId0 = timeSetEvent(4, 0, TimerFunction0, 0, TIME_PERIODIC);
    if(mmTimerId0 == (MMRESULT)0){
        printf("Failed to generate multimedia timer 0.\n");
        goto exit0;}
    Sleep(2);
    mmTimerId1 = timeSetEvent(4, 0, TimerFunction1, 0, TIME_PERIODIC);
    if(mmTimerId1 == (MMRESULT)0){
        printf("Failed to generate multimedia timer 1.\n");
        goto exit0;}
    Sleep(2048);                        /* wait 2.048 seconds */

exit0:
    if(mmTimerId0)
        timeKillEvent(mmTimerId0);
    if(mmTimerId1)
        timeKillEvent(mmTimerId1);
    timeEndPeriod(1);                   /* restore resolution to default */

    printf("dwTicks0 %d\n", dwTicks0);
    printf("dwTicks1 %d\n", dwTicks1);

exit1:
    return(0);
}

/*----------------------------------------------------------------------*/
/*      TimerFunction       increments dwTicks                          */
/*----------------------------------------------------------------------*/
void CALLBACK TimerFunction0(UINT uTimerID, UINT uMsg,
                        DWORD dwUser, DWORD dw1, DWORD dw2)
{
    dwTicks0 += 1;                           /* increment ticks */
}

/*----------------------------------------------------------------------*/
/*      TimerFunction       increments dwTicks                          */
/*----------------------------------------------------------------------*/
void CALLBACK TimerFunction1(UINT uTimerID, UINT uMsg,
                        DWORD dwUser, DWORD dw1, DWORD dw2)
{
    dwTicks1 += 1;                           /* increment ticks */
}

**rcgldr** · 05-04-2016

I compiled and tested the same code from post #21 using Visual Studio Express 2015 and Windows 7, and it runs OK, so having multiple setTimeEvent() isn't an issue for Win XP or Win 7.

Thread: win32 - timeSetEvent(): what is wrong with these function?

Thread Tools

Search Thread

Display

Similar Threads

timeSetEvent

win32 timer function

timeSetEvent problem

What's wrong with my Win32 Wrapper code?

compiling mingw to enable timeSetEvent