fixes -mmt= option for many handlers (previously ignored by handler…

…s used SetCoderProperties to set number of threads)
mcmilk · Aug 31, 2023 · a884f29 · a884f29
1 parent 8f52579
commit a884f29
Showing 1 changed file with 6 additions and 7 deletions.
diff --git a/CPP/7zip/Archive/Common/HandlerOut.cpp b/CPP/7zip/Archive/Common/HandlerOut.cpp
@@ -203,8 +203,9 @@ HRESULT CMultiMethodProps::SetProperty(const wchar_t *nameSpec, const PROPVARIAN
 
   {
     HRESULT hres;
-    if (SetCommonProperty(name, value, hres))
-      return hres;
+    SetCommonProperty(name, value, hres);
+    /* don't return here, since many handlers set common properties (e. g. kNumThreads)
+       with SetCoderProperties, so add it also as prop by its ID from name below */
   }
 
   UInt32 number;
@@ -258,11 +259,9 @@ HRESULT CSingleMethodProps::SetProperty(const wchar_t *name2, const PROPVARIANT
   }
   {
     HRESULT hres;
-    if (SetCommonProperty(name, value, hres))
-    {
-      // processed = true;
-      return S_OK;
-    }
+    SetCommonProperty(name, value, hres);
+    /* don't return here, since many handlers set common properties (e. g. kNumThreads)
+       with SetCoderProperties, so add it also as prop by its ID from name below */
   }
   RINOK(ParseMethodFromPROPVARIANT(name, value));
   return S_OK;