Jeremy White wrote:
I've just committed a change that added two new methods, GetParent and UserData - both of which can be called on controls or windows.

GetParent simply returns the parent window of the control or child window (if there was one).

Just one comment here - if the parent doesn't have valid userdata, should GetParent() return the hwnd, like most other calls do? Should we be looking at extending/changing other api calls to return objects, where such objects exist?

UserData allows you to associate an SV to a control or window, allowing you to save instance data to that object.

Nice. I hadn't thought of holding a separate reference to the userdata, and so avoiding the circular references. I think we may still see problems with the reference causing 'late destruction' in some cases, but I could be wrong (I still can't really get my head round the way objects are tied, resulting in more than one call to the classes DESTROY method).

I've created a test script for both of these functions but I am unsure of the naming scheme? I also assume that going forward we should be adding test scripts for any changes?

It would be nice to have tests for everything we add, and we should certainly try going forward. There are some things I just don't know how to test yet. I'd like to encourage everyone to put tests into the test directory - I've not really got a naming scheme yet, but think something will become obvious once we see some more contributions. For methods, I started numbering at 50, with test names XX_Classname_Methodname.t, but don't know if this will hold up or not.

Regards,
Rob.

Reply via email to