[PATCH v2 3/5] devenum: Make some variables static.

Zebediah Figura z.figura12 at gmail.com
Tue Jun 26 09:05:31 CDT 2018


Signed-off-by: Zebediah Figura <z.figura12 at gmail.com>
---
 dlls/devenum/createdevenum.c   | 15 +++++++--------
 dlls/devenum/devenum_main.c    | 13 ++++---------
 dlls/devenum/devenum_private.h |  4 +---
 dlls/devenum/mediacatenum.c    |  4 +++-
 4 files changed, 15 insertions(+), 21 deletions(-)

diff --git a/dlls/devenum/createdevenum.c b/dlls/devenum/createdevenum.c
index 06c264c..e3ae54f 100644
--- a/dlls/devenum/createdevenum.c
+++ b/dlls/devenum/createdevenum.c
@@ -41,8 +41,6 @@
 
 WINE_DEFAULT_DEBUG_CHANNEL(devenum);
 
-extern HINSTANCE DEVENUM_hInstance;
-
 static const WCHAR wszFilterKeyName[] = {'F','i','l','t','e','r',0};
 static const WCHAR wszMeritName[] = {'M','e','r','i','t',0};
 static const WCHAR wszPins[] = {'P','i','n','s',0};
@@ -424,6 +422,7 @@ static void register_legacy_filters(void)
             TRACE("Registering %s\n", debugstr_w(wszFilterSubkeyName));
 
             strcpyW(wszRegKey, clsidW);
+            strcatW(wszRegKey, backslashW);
             strcatW(wszRegKey, wszFilterSubkeyName);
 
             if (RegOpenKeyExW(HKEY_CLASSES_ROOT, wszRegKey, 0, KEY_READ, &classkey) != ERROR_SUCCESS)
@@ -460,7 +459,7 @@ static void register_legacy_filters(void)
             V_VT(&var) = VT_BSTR;
             if (!(V_BSTR(&var) = SysAllocString(wszFilterSubkeyName)))
                 goto cleanup;
-            hr = IPropertyBag_Write(prop_bag, clsid_keyname, &var);
+            hr = IPropertyBag_Write(prop_bag, clsidW, &var);
             if (FAILED(hr)) goto cleanup;
             VariantClear(&var);
 
@@ -537,7 +536,7 @@ static BOOL CALLBACK register_dsound_devices(GUID *guid, const WCHAR *desc, cons
     StringFromGUID2(&CLSID_DSoundRender, clsid, CHARS_IN_GUID);
     if (!(V_BSTR(&var) = SysAllocString(clsid)))
         goto cleanup;
-    hr = IPropertyBag_Write(prop_bag, clsid_keyname, &var);
+    hr = IPropertyBag_Write(prop_bag, clsidW, &var);
     if (FAILED(hr)) goto cleanup;
     VariantClear(&var);
 
@@ -620,7 +619,7 @@ static void register_waveout_devices(void)
         StringFromGUID2(&CLSID_AudioRender, clsid, CHARS_IN_GUID);
         if (!(V_BSTR(&var) = SysAllocString(clsid)))
             goto cleanup;
-        hr = IPropertyBag_Write(prop_bag, clsid_keyname, &var);
+        hr = IPropertyBag_Write(prop_bag, clsidW, &var);
         if (FAILED(hr)) goto cleanup;
         VariantClear(&var);
 
@@ -693,7 +692,7 @@ static void register_wavein_devices(void)
         StringFromGUID2(&CLSID_AudioRecord, clsid, CHARS_IN_GUID);
         if (!(V_BSTR(&var) = SysAllocString(clsid)))
             goto cleanup;
-        hr = IPropertyBag_Write(prop_bag, clsid_keyname, &var);
+        hr = IPropertyBag_Write(prop_bag, clsidW, &var);
         if (FAILED(hr)) goto cleanup;
         VariantClear(&var);
 
@@ -765,7 +764,7 @@ static void register_midiout_devices(void)
         StringFromGUID2(&CLSID_AVIMIDIRender, clsid, CHARS_IN_GUID);
         if (!(V_BSTR(&var) = SysAllocString(clsid)))
             goto cleanup;
-        hr = IPropertyBag_Write(prop_bag, clsid_keyname, &var);
+        hr = IPropertyBag_Write(prop_bag, clsidW, &var);
         if (FAILED(hr)) goto cleanup;
         VariantClear(&var);
 
@@ -854,7 +853,7 @@ static void register_vfw_codecs(void)
         StringFromGUID2(&CLSID_AVICo, clsid, CHARS_IN_GUID);
         if (!(V_BSTR(&var) = SysAllocString(clsid)))
             goto cleanup;
-        hr = IPropertyBag_Write(prop_bag, clsid_keyname, &var);
+        hr = IPropertyBag_Write(prop_bag, clsidW, &var);
         if (FAILED(hr)) goto cleanup;
         VariantClear(&var);
 
diff --git a/dlls/devenum/devenum_main.c b/dlls/devenum/devenum_main.c
index 3499f99..98ad1ad 100644
--- a/dlls/devenum/devenum_main.c
+++ b/dlls/devenum/devenum_main.c
@@ -26,7 +26,7 @@
 WINE_DEFAULT_DEBUG_CHANNEL(devenum);
 
 DECLSPEC_HIDDEN LONG dll_refs;
-DECLSPEC_HIDDEN HINSTANCE DEVENUM_hInstance;
+static HINSTANCE devenum_instance;
 
 typedef struct
 {
@@ -36,11 +36,6 @@ typedef struct
 } register_info;
 
 /***********************************************************************
- *		Global string constant definitions
- */
-const WCHAR clsid_keyname[6] = { 'C', 'L', 'S', 'I', 'D', 0 };
-
-/***********************************************************************
  *		DllEntryPoint
  */
 BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD fdwReason, LPVOID fImpLoad)
@@ -49,7 +44,7 @@ BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD fdwReason, LPVOID fImpLoad)
 
     switch(fdwReason) {
     case DLL_PROCESS_ATTACH:
-        DEVENUM_hInstance = hinstDLL;
+        devenum_instance = hinstDLL;
         DisableThreadLibraryCalls(hinstDLL);
 	break;
     }
@@ -166,7 +161,7 @@ HRESULT WINAPI DllRegisterServer(void)
 
     TRACE("\n");
 
-    res = __wine_register_resources( DEVENUM_hInstance );
+    res = __wine_register_resources( devenum_instance );
     if (FAILED(res))
         return res;
 
@@ -214,5 +209,5 @@ HRESULT WINAPI DllRegisterServer(void)
 HRESULT WINAPI DllUnregisterServer(void)
 {
     FIXME("stub!\n");
-    return __wine_unregister_resources( DEVENUM_hInstance );
+    return __wine_unregister_resources( devenum_instance );
 }
diff --git a/dlls/devenum/devenum_private.h b/dlls/devenum/devenum_private.h
index ea4e017..e18c57d 100644
--- a/dlls/devenum/devenum_private.h
+++ b/dlls/devenum/devenum_private.h
@@ -83,7 +83,7 @@ extern IParseDisplayName DEVENUM_ParseDisplayName DECLSPEC_HIDDEN;
  */
 
 static const WCHAR backslashW[] = {'\\',0};
-static const WCHAR clsidW[] = {'C','L','S','I','D','\\',0};
+static const WCHAR clsidW[] = {'C','L','S','I','D',0};
 static const WCHAR instanceW[] = {'\\','I','n','s','t','a','n','c','e',0};
 static const WCHAR wszActiveMovieKey[] = {'S','o','f','t','w','a','r','e','\\',
                                           'M','i','c','r','o','s','o','f','t','\\',
@@ -93,5 +93,3 @@ static const WCHAR deviceW[] = {'@','d','e','v','i','c','e',':',0};
 static const WCHAR dmoW[] = {'d','m','o',':',0};
 static const WCHAR swW[] = {'s','w',':',0};
 static const WCHAR cmW[] = {'c','m',':',0};
-
-extern const WCHAR clsid_keyname[6] DECLSPEC_HIDDEN;
diff --git a/dlls/devenum/mediacatenum.c b/dlls/devenum/mediacatenum.c
index c4338a2..660e81c 100644
--- a/dlls/devenum/mediacatenum.c
+++ b/dlls/devenum/mediacatenum.c
@@ -347,6 +347,7 @@ static HRESULT create_PropertyBag(MediaCatMoniker *mon, IPropertyBag **ppBag)
     else if (rpb->type == DEVICE_FILTER)
     {
         strcpyW(rpb->path, clsidW);
+        strcatW(rpb->path, backslashW);
         if (mon->has_class)
         {
             StringFromGUID2(&mon->class, rpb->path + strlenW(rpb->path), CHARS_IN_GUID);
@@ -496,7 +497,7 @@ static HRESULT WINAPI DEVENUM_IMediaCatMoniker_BindToObject(IMoniker *iface, IBi
             if (SUCCEEDED(res))
             {
                 V_VT(&var) = VT_LPWSTR;
-                res = IPropertyBag_Read(pProp, clsid_keyname, &var, NULL);
+                res = IPropertyBag_Read(pProp, clsidW, &var, NULL);
             }
             if (SUCCEEDED(res))
             {
@@ -1027,6 +1028,7 @@ HRESULT create_EnumMoniker(REFCLSID class, IEnumMoniker **ppEnumMoniker)
     pEnumMoniker->class = *class;
 
     strcpyW(buffer, clsidW);
+    strcatW(buffer, backslashW);
     StringFromGUID2(class, buffer + strlenW(buffer), CHARS_IN_GUID);
     strcatW(buffer, instanceW);
     if (RegOpenKeyExW(HKEY_CLASSES_ROOT, buffer, 0, KEY_ENUMERATE_SUB_KEYS, &pEnumMoniker->sw_key))
-- 
2.7.4




More information about the wine-devel mailing list