[PATCH] winetest: Add GUI to display results of individual tests
11 Dec
11 Dec
7:52 p.m.
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.
8:44 p.m.
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.
2565
Age (days ago)
2566
Last active (days ago)
2 comments
3 participants
participants (3)
-
André Hentschel -
Isira Seneviratne -
Zebediah Figura