Jinoh Kang (@iamahuman) commented about dlls/comctl32/tests/misc.c:
FreeLibrary(hinst);
}
+#define SUPER_CLASS_NAME_A "SuperClass Test" +static WNDPROC real_class_wndproc; +static LRESULT WINAPI super_class_test_win_proc_a(HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam) +{
- if (real_class_wndproc)
CallWindowProcA(real_class_wndproc, hwnd, msg, wparam, lparam);
I don't think calling real wndproc *and then* calling defwndproc is a good idea, because the real wndproc can end up calling defwndproc anyway and this results in double operations.
```suggestion:-0+0 return CallWindowProcA(real_class_wndproc, hwnd, msg, wparam, lparam); ```
Otherwise, can you leave a comment as to why it had to be done this way?