Re: comctl32: Shed an unused parameter of TOOLTIPS_HitTestT and rename to TOOLTIPS_HitTest.

[James McKenzie]

On 2/27/11 1:46 AM, Nikolay Sivov wrote:
On 2/26/2011 23:01, Gerald Pfeifer wrote:
---
  dlls/comctl32/tooltips.c |    7 +++----
  1 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/dlls/comctl32/tooltips.c b/dlls/comctl32/tooltips.c
index a2e58e0..445845c 100644
--- a/dlls/comctl32/tooltips.c
+++ b/dlls/comctl32/tooltips.c
@@ -1396,8 +1396,7 @@ TOOLTIPS_GetToolInfoT (const TOOLTIPS_INFO 
*infoPtr, TTTOOLINFOW *ti, BOOL isW)


  static LRESULT
-TOOLTIPS_HitTestT (const TOOLTIPS_INFO *infoPtr, LPTTHITTESTINFOW 
lptthit,
-                   BOOL isW)
+TOOLTIPS_HitTest (const TOOLTIPS_INFO *infoPtr, LPTTHITTESTINFOW 
lptthit)
  {
      TTTOOL_INFO *toolPtr;
      INT nTool;
@@ -2209,8 +2208,8 @@ TOOLTIPS_WindowProc (HWND hwnd, UINT uMsg, 
WPARAM wParam, LPARAM lParam)

      case TTM_HITTESTA:
      case TTM_HITTESTW:
-        return TOOLTIPS_HitTestT (infoPtr, (LPTTHITTESTINFOW)lParam,
-                                      uMsg == TTM_HITTESTW);
+        return TOOLTIPS_HitTest (infoPtr, (LPTTHITTESTINFOW)lParam);
+
      case TTM_NEWTOOLRECTA:
      case TTM_NEWTOOLRECTW:
          return TOOLTIPS_NewToolRectT (infoPtr, (LPTTTOOLINFOW)lParam);
This should properly handle text instead of removing A/W case.


Even if all it does is convert ANSI to UNICODE and then call the UNICODE 
case correct?

This is more a question on how I should handle a case I'm working with 
for riched20, not a comment on this particular piece of code.

James McKenzie