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.
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.