Hi Pritpal, >> hbqt_destruct.cpp:92: return hb_parptr( iParam ); /* TOFIX: This is > dangerous. */ > > Now the pointer type is checked, so I think it is the only way to control > it. > So I have removed TOFIX.
Ops, pls see my other mail. I think this is still wrong and for multiple reasons. HBQT needs an error handler in the first place and RTEs, not GPFs. GPFs are pretty difficult to track down for most programmers. >> hbqt_events.cpp:118: /* TOFIX: Possible leak of PHB_ITEMs stored in list. > */ > > Fixed > >> hbqt_hbqtableview.cpp:196:/* TOFIX: Leak */ > > Fixed. > >> hbqt_slots.cpp:110: /* TOFIX: Possible leak of PHB_ITEMs stored in list. > > Fixed. Thank you, these look good. >> As for proper error handling I think HBQT would need >> an errorsys replacement which uses QT MessageBox to >> popup RTE information. >> > > Yes sure, first we need to introduce RTE mechanism here. Yes. Brgds, Viktor _______________________________________________ Harbour mailing list (attachment size limit: 40KB) [email protected] http://lists.harbour-project.org/mailman/listinfo/harbour
