Re: regedit: write support for strings

[Steven Edwards]

Hello Dimi,

--- "Dimitrie O. Paun"  wrote:
> I figured I should so something a bit more "useful" (in light of
> the 0.9 release), so here is the first attempt. It's not too much,
> and not too good, but after looking at the code, it needs some
> major cleanup before I can do any work on it. So the plan is:
>   -- get this in the tree, so I can cleanup my tree
>   -- cleanup the code (proper indentation, etc)
>   -- add more edit support to it.
>
> ChangeLog
>     Add support for editing strings.

The ReactOS and WINE regedits are a little out of sync. Can you
incorporate this in to your next patch?

Thanks
Steven

Changelog:
Filip Navara
- Fixed some flags in calls to CreateWindowEx
- Fixed Color of the splitter (should be COLOR_BTNFACE and not light
gray brush)
Steven Edwards
- Change a few DbgPrint statements to WINE_TRACE

__________________________________
Do you Yahoo!?
Free Pop-Up Blocker - Get it now
http://companion.yahoo.com/

regedit: write support for strings

[Dimitrie O. Paun]

On December 1, 2003 06:29 pm, Steven Edwards wrote:
> The ReactOS and WINE regedits are a little out of sync. Can you
> incorporate this in to your next patch?

OK,

but the bulk of it is this:

+struct {
+   char *Name;
+   ULONG Id;
+} Messages[] =
+{
+   {"WM_ACTIVATE", WM_ACTIVATE},
...
+   {"WM_MOUSELEAVE", 0x2A3}
+};  
+   
 /*******************************************************************************
  *
  *  FUNCTION: ChildWndProc(HWND, unsigned, WORD, LONG)
@@ -129,6 +321,25 @@
     static int last_split;
 /*    ChildWnd* pChildWnd = (ChildWnd*)GetWindowLong(hWnd, GWL_USERDATA); */
     static ChildWnd* pChildWnd;
+
+        {
+           int i;
+           char *name = NULL;
+
+           for (i = sizeof(Messages) / sizeof(Messages[0]); i--; )
+              if (message == Messages[i].Id)
+              {
+                 name = Messages[i].Name;
+                 break;
+              }
+
+           if (name)
+              WINE_TRACE("Window: %x Message: %s (%x) wParam: %x lParam: %x\n",
+                 hWnd, name, message, wParam, lParam);
+           else
+              WINE_TRACE("Window: %x Message: %x wParam: %x lParam: %x\n",
+                 hWnd, message, wParam, lParam);
+        }

which I must say has no business being in here, for too many reasons to
list here. This part has to go.