On 07/05/12 18:31, Erich E. Hoover wrote:
On Thu, Jul 5, 2012 at 3:09 AM, Jacek Caban jacek@codeweavers.com wrote:
... While it's not about this particular patch, I can see that you're changing hhctrl.ocx to the point where it has an useful API (as opposed to simple hh.exe-like help viewer). It is really nice to see, thanks. It would be great to have some tests for this. Please consider adding them (I'm sure you already have some code for testing, so I imagine it would be mostly integrating it with Wine test suite).
Once the get/set HH_WINTYPE commands are in then the only thing that remains in order to construct tests is the ability to build CHM files. I've been working on trying to get HTML Help Workshop to be able to compile CHM files, but so far this endeavor has not met with much success.
We already have a trivial .chm file in itss tests. It's compiled by HTML Help Workshop on Windows. You probably could do the same for hhctrl.ocx. Just make sure others will be able to reproduce the file if needed.
Did you see anything in these patches that you would like changed? I just noticed a double-free mistake, so I'm going to resubmit with that fixed when I get a chance and can include any suggestions you might have.
Sorry I couldn't get to this sooner. I will review them in a moment.
Jacek