Commits

Adrian Buehlmann committed 44714ca

shellext: use S_OK instead of NOERROR

Not knowingly fixing a bug, but S_OK seems to be the preferred style

Comments (0)

Files changed (3)

win32/shellext/CShellExtCMenu.cpp

         bAppendItems = FALSE;
 
     if (!bAppendItems)
-        return NOERROR;
+        return S_OK;
 
     const std::size_t sz = sizeof(menuDescList) / sizeof(MenuDescription);
     bool promoted[sz];
         if (iter != MenuIdMap.end())
         {
             DoHgtk(iter->second.name);
-            hr = NOERROR;
+            hr = S_OK;
         }
         else
         {
         break;
 
     default:
-        return NOERROR;
+        return S_OK;
     }
 
-    return NOERROR;
+    return S_OK;
 }
 
 
         myFolder = name;
     }
 
-    return NOERROR;
+    return S_OK;
 }
 
 
     if (*ppv)
     {
         AddRef();
-        return NOERROR;
+        return S_OK;
     }
 
     return E_NOINTERFACE;

win32/shellext/CShellExtOverlay.cpp

     if (*ppv)
     {
         AddRef();
-        return NOERROR;
+        return S_OK;
     }
 
     return E_NOINTERFACE;

win32/shellext/ThgClassFactory.h

         {
             *ppv = (LPCLASSFACTORY) this;
             AddRef();
-            return NOERROR;
+            return S_OK;
         }
 
         return E_NOINTERFACE;
 
     STDMETHODIMP LockServer(BOOL fLock)
     {
-        return NOERROR;
+        return S_OK;
     }
 
 };