Don't need the MAKEINTRESOURCE.
That is a macro to convert between the int numbers that corespond to control ID's and the names you have declared.
In this case you do not need it as you have not given them a name only a number.
If you have a dialog with more controls you will get into trouble in your callback, or if someone else has to debug your code.
Try to find the correct case in the WM_COMMAND switch when they are all just numbers is a pain and is not good style. Call the edits say ID_POINT_X (not 102) and I would know what I was looking for.
The edits are called 102 and 103 in your code.
This is the second paramater in the call to GetDlgItemInt() eg pointX and pointY
Do not use pointers to get the return.
PS SelectObject() has a return. A piece of memory.
Catch the return for later use as it is important. If you do not your PC will crash from memory loss. May take a while as your leak is only small, two brushes per paint msg.
Its something you will have to learn to do sooner or later.
