[PATCH] winetest: Add GUI to display results of individual tests

Zebediah Figura z.figura12 at gmail.com
Tue Dec 11 14:44:36 CST 2018


On 12/11/2018 01:52 PM, André Hentschel wrote:
> Hi,
> 
> I like the look, but didn't reviewed it and the code completely.
> Regarding the code I think we can get rid of displayResults, also IDD_RESULT_LIST seems unused.
> You can stay within a line length of 100, so some line breaks are not needed.
> 
> 

I haven't done a thorough review yet either, but I guess the first thing 
I'd suggest is perhaps adding this listview to the existing dialog 
window, rather than creating a new one.



More information about the wine-devel mailing list