<div dir="ltr"><div><div><div>Hi, <br><br></div>Thanks for your patch! <br></div>I see there are still many style inconsistencies here, like "foo=bar;" and "foo = bar;" mixed together. Same for passing arguments, like "foo(bar,baz)" and "foo(bar, baz)" mixed together. Please try sticking to a single convention, it helps in code readability. There also might be an issue with the consistency of indendation on longer lines which do not fit. As long as you keep the indentation levels consistent across the file you should be fine.<br><br></div><div><div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Aug 27, 2016 at 1:56 PM, Ruslan Kabatsayev <span dir="ltr"><<a href="mailto:b7.10110111@gmail.com" target="_blank">b7.10110111@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+INT_PTR CALLBACK system_tab_proc(HWND hsystab, UINT msg, WPARAM wparam, LPARAM lparam)<br>
+{<br>
+    switch(msg)<br>
+    {<br>
+    case WM_COMMAND:<br>
+        switch(HIWORD(wparam))<br>
+        {<br>
+        case BN_CLICKED:<br>
+            switch(wparam)<br>
+            {<br>
+            case IDC_WHQL_CHECKBOX:<br>
+            {<br>
+                update_dxdiag_info();<br>
+                return TRUE;<br>
+            }<br>
+            }<br>
+            break;<br></blockquote><div>There's an indentation issue here. <br></div></div><br></div><div class="gmail_extra">You should still wait for others to review this before re-sending this patch though.<br><br></div><div class="gmail_extra">Cheers,<br></div><div class="gmail_extra">Aaryaman<br></div></div></div></div></div></div>