[akhaldi] 73337: [COMDLG32] Sync with Wine Staging 1.9.23. CORE-12409

Author: akhaldi
Date: Tue Nov 22 12:15:51 2016
New Revision: 73337

URL: http://svn.reactos.org/svn/reactos?rev=73337&view=rev
Log:
[COMDLG32] Sync with Wine Staging 1.9.23. CORE-12409

Modified:
    trunk/reactos/dll/win32/comdlg32/colordlg.c
    trunk/reactos/dll/win32/comdlg32/comdlg32_ros.diff
    trunk/reactos/dll/win32/comdlg32/filedlg.c
    trunk/reactos/dll/win32/comdlg32/fontdlg.c
    trunk/reactos/media/doc/README.WINE

Modified: trunk/reactos/dll/win32/comdlg32/colordlg.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comdlg32/colordlg...
==============================================================================
--- trunk/reactos/dll/win32/comdlg32/colordlg.c	[iso-8859-1] (original)
+++ trunk/reactos/dll/win32/comdlg32/colordlg.c	[iso-8859-1] Tue Nov 22 12:15:51 2016
@@ -661,15 +661,11 @@
    int r = GetRValue(cr);
    int g = GetGValue(cr);
    int b = GetBValue(cr);
-   char buffer[10];

infoPtr->updating = TRUE;
-   sprintf(buffer, "%d", r);
-   SetWindowTextA( GetDlgItem(infoPtr->hwndSelf, IDC_COLOR_EDIT_R), buffer);
-   sprintf(buffer, "%d", g);
-   SetWindowTextA( GetDlgItem(infoPtr->hwndSelf, IDC_COLOR_EDIT_G), buffer);
-   sprintf( buffer, "%d", b );
-   SetWindowTextA( GetDlgItem(infoPtr->hwndSelf, IDC_COLOR_EDIT_B), buffer);
+   SetDlgItemInt(infoPtr->hwndSelf, IDC_COLOR_EDIT_R, r, TRUE);
+   SetDlgItemInt(infoPtr->hwndSelf, IDC_COLOR_EDIT_G, g, TRUE);
+   SetDlgItemInt(infoPtr->hwndSelf, IDC_COLOR_EDIT_B, b, TRUE);
    infoPtr->updating = FALSE;
  }
 }
@@ -681,15 +677,10 @@
 {
  if (IsWindowVisible( GetDlgItem(infoPtr->hwndSelf, IDC_COLOR_GRAPH) ))   /* if full size */
  {
-   char buffer[10];
-
    infoPtr->updating = TRUE;
-   sprintf(buffer, "%d", infoPtr->h);
-   SetWindowTextA( GetDlgItem(infoPtr->hwndSelf, IDC_COLOR_EDIT_H), buffer);
-   sprintf(buffer, "%d", infoPtr->s);
-   SetWindowTextA( GetDlgItem(infoPtr->hwndSelf, IDC_COLOR_EDIT_S), buffer);
-   sprintf(buffer, "%d", infoPtr->l);
-   SetWindowTextA( GetDlgItem(infoPtr->hwndSelf, IDC_COLOR_EDIT_L), buffer);
+   SetDlgItemInt(infoPtr->hwndSelf, IDC_COLOR_EDIT_H, infoPtr->h, TRUE);
+   SetDlgItemInt(infoPtr->hwndSelf, IDC_COLOR_EDIT_S, infoPtr->s, TRUE);
+   SetDlgItemInt(infoPtr->hwndSelf, IDC_COLOR_EDIT_L, infoPtr->l, TRUE);
    infoPtr->updating = FALSE;
  }
  CC_PaintLumBar(infoPtr);

Modified: trunk/reactos/dll/win32/comdlg32/comdlg32_ros.diff
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comdlg32/comdlg32...
==============================================================================
--- trunk/reactos/dll/win32/comdlg32/comdlg32_ros.diff	[iso-8859-1] (original)
+++ trunk/reactos/dll/win32/comdlg32/comdlg32_ros.diff	[iso-8859-1] Tue Nov 22 12:15:51 2016
@@ -1,7 +1,7 @@
 diff -pudN e:\wine\dlls\comdlg32/cdlg32.c e:\reactos\dll\win32\comdlg32/cdlg32.c
---- e:\wine\dlls\comdlg32/cdlg32.c	2015-02-21 17:13:08.621542200 +0100
-+++ e:\reactos\dll\win32\comdlg32/cdlg32.c	2015-07-21 09:48:05.584922500 +0100
-@@ -174,6 +157,8 @@ DWORD WINAPI CommDlgExtendedError(void)
+--- e:\wine\dlls\comdlg32/cdlg32.c	2016-11-16 17:28:42 +0100
++++ e:\reactos\dll\win32\comdlg32/cdlg32.c	2015-08-27 22:02:19 +0100
+@@ -157,6 +157,8 @@ DWORD WINAPI CommDlgExtendedError(void)
      return 0; /* we never set an error, so there isn't one */
  }

@@ -10,7 +10,7 @@
  /*************************************************************************
   * Implement the CommDlg32 class factory
   *
-@@ -283,7 +268,11 @@ HRESULT WINAPI DllGetClassObject(REFCLSI
+@@ -266,7 +268,11 @@ HRESULT WINAPI DllGetClassObject(REFCLSI
   */
  HRESULT WINAPI DllRegisterServer(void)
  {
@@ -22,7 +22,7 @@
  }

/***********************************************************************
-@@ -291,5 +280,11 @@ HRESULT WINAPI DllRegisterServer(void)
+@@ -274,5 +280,11 @@ HRESULT WINAPI DllRegisterServer(void)
   */
  HRESULT WINAPI DllUnregisterServer(void)
  {
@@ -34,27 +34,10 @@
  }
 +
 +#endif /* Win 7 */
-
-diff -pudN e:\wine\dlls\comdlg32/comdlg32.spec e:\reactos\dll\win32\comdlg32/comdlg32.spec
---- e:\wine\dlls\comdlg32/comdlg32.spec	2015-02-21 17:13:08.621542200 +0100
-+++ e:\reactos\dll\win32\comdlg32/comdlg32.spec	2012-02-15 22:32:22.456422900 +0100
-@@ -3,9 +3,9 @@
- @ stdcall ChooseFontA(ptr)
- @ stdcall ChooseFontW(ptr)
- @ stdcall CommDlgExtendedError()
--@ stdcall -private DllGetClassObject(ptr ptr ptr)
--@ stdcall -private DllRegisterServer()
--@ stdcall -private DllUnregisterServer()
-+;@ stdcall -private DllGetClassObject(ptr ptr ptr) ; Win 7
-+;@ stdcall -private DllRegisterServer() ; Win 7
-+;@ stdcall -private DllUnregisterServer() ; Win 7
- @ stdcall FindTextA(ptr)
- @ stdcall FindTextW(ptr)
- @ stdcall GetFileTitleA(str ptr long)
 diff -pudN e:\wine\dlls\comdlg32/itemdlg.c e:\reactos\dll\win32\comdlg32/itemdlg.c
---- e:\wine\dlls\comdlg32/itemdlg.c	2015-04-05 20:44:56.807083800 +0100
-+++ e:\reactos\dll\win32\comdlg32/itemdlg.c	2015-07-21 09:56:34.329021000 +0100
-@@ -18,10 +18,13 @@
+--- e:\wine\dlls\comdlg32/itemdlg.c	2016-08-14 19:11:13 +0100
++++ e:\reactos\dll\win32\comdlg32/itemdlg.c	2016-08-15 16:16:18 +0100
+@@ -18,6 +18,8 @@
   * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA
   */

@@ -63,12 +46,7 @@
  #include <stdarg.h>

#define COBJMACROS
- #define NONAMELESSUNION
-+#define NONAMELESSSTRUCT
- 
- #include "windef.h"
- #include "winbase.h"
-@@ -3847,3 +3850,5 @@ HRESULT FileSaveDialog_Constructor(IUnkn
+@@ -4659,3 +4661,5 @@ HRESULT FileSaveDialog_Constructor(IUnkn
  {
      return FileDialog_constructor(pUnkOuter, riid, ppv, ITEMDLG_TYPE_SAVE);
  }

Modified: trunk/reactos/dll/win32/comdlg32/filedlg.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comdlg32/filedlg....
==============================================================================
--- trunk/reactos/dll/win32/comdlg32/filedlg.c	[iso-8859-1] (original)
+++ trunk/reactos/dll/win32/comdlg32/filedlg.c	[iso-8859-1] Tue Nov 22 12:15:51 2016
@@ -1550,10 +1550,16 @@
          result = GetFullPathNameW(fodInfos->filename, MAX_PATH, tmpBuf, &nameBit);
          if (result) {

-            /* nameBit is always shorter than the original filename */
-            lstrcpyW(fodInfos->filename,nameBit);
-
-            *nameBit = 0x00;
+            /* nameBit is always shorter than the original filename. It may be NULL
+             * when the filename contains only a drive name instead of file name */
+            if (nameBit)
+            {
+                lstrcpyW(fodInfos->filename,nameBit);
+                *nameBit = 0x00;
+            }
+            else
+                *fodInfos->filename = '\0';
+
             MemFree(fodInfos->initdir);
             fodInfos->initdir = MemAlloc((lstrlenW(tmpBuf) + 1)*sizeof(WCHAR));
             lstrcpyW(fodInfos->initdir, tmpBuf);
@@ -2526,6 +2532,7 @@
                 && fodInfos->ofnInfos->Flags & OFN_EXPLORER)
             {
               SendCustomDlgNotificationMessage(hwnd, CDN_FOLDERCHANGE);
+              SendMessageA(fodInfos->DlgInfos.hwndFileName, WM_SETTEXT, 0, (LPARAM)"");
             }
      }
      else if( nOpenAction == ONOPEN_SEARCH )
@@ -2536,6 +2543,13 @@
           COMDLG32_SHFree(pidlCurrent);
           if (filename_is_edit( fodInfos ))
               SendMessageW(fodInfos->DlgInfos.hwndFileName, EM_SETSEL, 0, -1);
+          else
+          {
+              HWND hwnd;
+
+              hwnd = (HWND)SendMessageA(fodInfos->DlgInfos.hwndFileName, CBEM_GETEDITCONTROL, 0, 0);
+              SendMessageW(hwnd, EM_SETSEL, 0, -1);
+          }
         }
       }
       ret = FALSE;

Modified: trunk/reactos/dll/win32/comdlg32/fontdlg.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comdlg32/fontdlg....
==============================================================================
--- trunk/reactos/dll/win32/comdlg32/fontdlg.c	[iso-8859-1] (original)
+++ trunk/reactos/dll/win32/comdlg32/fontdlg.c	[iso-8859-1] Tue Nov 22 12:15:51 2016
@@ -517,7 +517,6 @@

static void CFn_FitFontSize( HWND hDlg, int points)
 {
-    WCHAR buffW[16];
     int i,n;

/* look for fitting font size in combobox3 */
@@ -536,8 +535,7 @@
     }

/* no default matching size, set text manually */
-    sprintfW(buffW, fontsizefmtW, points);
-    SetDlgItemTextW(hDlg, cmb3, buffW);
+    SetDlgItemInt(hDlg, cmb3, points, TRUE);
 }

static BOOL CFn_FitFontStyle( HWND hDlg, LONG packedstyle )

Modified: trunk/reactos/media/doc/README.WINE
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/doc/README.WINE?rev=7...
==============================================================================
--- trunk/reactos/media/doc/README.WINE	[iso-8859-1] (original)
+++ trunk/reactos/media/doc/README.WINE	[iso-8859-1] Tue Nov 22 12:15:51 2016
@@ -55,7 +55,7 @@
 reactos/dll/win32/clusapi             # Synced to WineStaging-1.9.11
 reactos/dll/win32/comcat              # Synced to WineStaging-1.9.11
 reactos/dll/win32/comctl32            # Synced to WineStaging-1.9.23
-reactos/dll/win32/comdlg32            # Synced to WineStaging-1.9.16
+reactos/dll/win32/comdlg32            # Synced to WineStaging-1.9.23
 reactos/dll/win32/compstui            # Synced to WineStaging-1.9.11
 reactos/dll/win32/credui              # Synced to WineStaging-1.9.16
 reactos/dll/win32/crypt32             # Synced to WineStaging-1.9.16