nanogui: Thread: i found a bug in nanox

I think there has some problems in  Microwindows-9.1 -->winuser.c-->MwDestroyWindow()
line465,as below:
   



 /* Remove all messages from msg queue for this window*/
    for(p=mwMsgHead.head; p; ) {
        pmsg = GdItemAddr(p, MSG, link);
        if(pmsg->hwnd == wp) {
            p = p->next;
            GdListRemove(&mwMsgHead, &pmsg->link);
            GdItemFree(p);   //this line is wrong!!!!!!
        } else
            p = p->next;
    }


" GdItemFree(p);" should be "GdItemFree(pmsg)"!!

Am i right?

> " GdItemFree(p);" should be "GdItemFree(pmsg)"!!

Yes - you're right.  This has been fixed in CVS for some
time.  If you're running win32 API, the CVS version has
quite a few enhancements, including support for dialogs.

Regards,

Greg

On Friday 09 December 2005 12:08, alex wrote:
> I think there has some problems in  Microwindows-9.1
> -->winuser.c-->MwDestroyWindow() line465,as below:
>
>
>
>
>  /* Remove all messages from msg queue for this window*/
>     for(p=mwMsgHead.head; p; ) {
>         pmsg = GdItemAddr(p, MSG, link);
>         if(pmsg->hwnd == wp) {
>             p = p->next;
>             GdListRemove(&mwMsgHead, &pmsg->link);
>             GdItemFree(p);   //this line is wrong!!!!!!
>         } else
>             p = p->next;
>     }
>
>
> " GdItemFree(p);" should be "GdItemFree(pmsg)"!!

I think it is already fixed in current cvs.

Bye
Alex
-- 
Work: ####@####.#### - http://www.jenoptik-los.de
Home: ####@####.####                - http://www.kde.org
      ####@####.####               - http://www.neundorf.net