how can i detect memory leaks?

[joaquim]

i have 1 nice tools\class's for use win32. but when i execute the application, time by time, the memory used is increased. and know that these is a memory leak. what i don't know is how to know where is the problem, on code
can anyone advice me please?

[MutantJohn]

This might help : c - Is there a good Valgrind substitute for Windows? - Stack Overflow

[Elysia]

Consider also using smart pointers.

[laserlight]

I suggest that you read Stroustrup's answer to the FAQ How do I deal with memory leaks?

[joaquim]

i belive that my memory leak is here:

else if(wParam==JoystickTimer)
                    {
                        JOYINFOEX  b={0};
                        b.dwSize=sizeof(JOYINFOEX );
                        b.dwFlags=JOY_RETURNALL;

                        //cicle all possible joysticks
                        for (int i=0; i<16; i++)
                        {
                            int direction=0;
                            if(joyGetPosEx(i,&b)== JOYERR_NOERROR) //if theres any error then continue to next joystick
                            {
                                if(i==0)
                                    inst->Joystick(-1, -1,-1);
                                else
                                    inst->Joystick(0, -1,-1);
                                break;
                            }

                            long h =b.dwButtons;

                            //testing the directions
                            //will be 8 directions(inclued diagonals)


                            if (b.dwXpos == 0 && b.dwYpos ==65535)
                            {
                                direction=5;
                            }
                            else if (b.dwXpos == 65535 && b.dwYpos ==0)
                            {
                                direction=6;
                            }
                            else if (b.dwXpos == 65535 && b.dwYpos ==65535)
                            {
                                direction=7;
                            }
                            else if(b.dwXpos == 0 && b.dwYpos == 0)
                            {
                                direction=8;
                            }

                            else if (b.dwXpos == 0)
                            {
                                direction=1;
                            }
                            else if (b.dwXpos == 65535)
                            {
                                direction=2;
                            }
                            else if (b.dwYpos == 0)
                            {
                                direction=3;
                            }
                            else if (b.dwYpos == 65535)
                            {
                                direction=4;
                            }
                            inst->Joystick(i, direction,h); //lambda function
                        }
                    }

- the form is a subclass window(like my others controls). the inst is the pointer to the form objects\instances.
why i think that it's these 'if'? i comment it, and i don't see the memory leak.
so how can i detect how many Joysticks are connected to pc?

[Elkvis]

You seem to be using "magic numbers" a lot in your program. I suggest creating suitable named constants/macros to use in place of your direction and possibly other values in your program.

What does joyGetPosEx do? Is it your function? Win32? What does the JOYINFOEX structure look like? I don't see any memory allocation happening in the code that you posted, so I don't think that's the source of your memory leak, if indeed one exists.

[joaquim]

You seem to be using "magic numbers" a lot in your program. I suggest creating suitable named constants/macros to use in place of your direction and possibly other values in your program.

What does joyGetPosEx do? Is it your function? Win32? What does the JOYINFOEX structure look like? I don't see any memory allocation happening in the code that you posted, so I don't think that's the source of your memory leak, if indeed one exists.

Moderator: i have found the memory leak. is about win32 Joystick function. so, if you can, change these topic to Windows subforum.. thanks.

joyGetPosEx() is for get the joystick information. but if i don't have a joystick connected to pc, these function give me a memory leak.
so i need test more and do a nice function for detect how many joysticks are connected to pc

[laserlight]

*moved as requested*