Hi Dave,
I had some problems with the applying the patch because it seems corrupted with characters ^[[m at the beginning and end or lines of the patch code. Once I removed them the patch applied properly.
The patch is working as expected and I have tested in all scenarios that I could think of. While reviewing I have only one doubt:
Should'nt we remove the definition and declaration of the OnCopy() function from ctlSQLGrid [ void ctlSQLGrid::OnCopy(wxCommandEvent &ev) ] as it is no longer being called, owning to the removal of the event trigger in the patch?
- EVT_MENU(MNU_COPY, ctlSQLGrid::OnCopy)
Apart from this the code looks good to me.
Thanks,