🚀 go-pugleaf

"Jan" <jan@jlcsoft.com> wrote:

>I have rewrite the code a little bit, and all the functions returns
>true, any idea what is wrong ?
>
>         RECT rc;
>         POINT pt;
>
>         hTarget = (HWND)lPreviewHandle;
>         Tmp = IsWindow(hTarget);
>         if(Tmp)
>          logtxt("Target = valid");
>         else
>          logtxt("Target = Invalid");
>
>         logtxt("Client window");
>         GetClientRect(hTarget, &rc);
>         logtxt("Top: " + IntToStr(rc.top));
>         logtxt("Left: " + IntToStr(rc.left));
>         logtxt("Right: " + IntToStr(rc.right));
>         logtxt("Bottom: " + IntToStr(rc.bottom));

So far, so good

>         Tmp = ClientToScreen(hTarget, &pt);

Where did pt come from? It's an uninitialised variable. You want to set
it to the top and left coordinates of the rectangle (well, to (0, 0).

>         if(Tmp) {
>          logtxt("ClientToScreen X: " + IntToStr(pt.x));
>          logtxt("ClientToScreen Y: " + IntToStr(pt.y));
>         } else {
>          logtxt("ClientToScreen = false");
>         }

As you saw, garbage out.

Do the ClientToScreen, and add the resulting X and Y values to the
rectangle (so top += Y, bottom += Y, left += X and right += X). You may
then get the resulting rectangle in the right place.

Alan Bellingham
-- 
alanb@episys.com