Fix setting of common properties (-mmt=n option is ignored by many handlers)

[sebres]

This PR fixes setting of -mmt=n option for many handlers (previously ignored by zstd, brotli, lz*, etc), so adding -mmt=1 to the call like below caused multi-threaded processing with default number of threads:

7z a -tzstd -mmt=1 archive.zstd large-file-to-compress.txt

So parallel compression of multiple files with several processes can cause unneeded context switches due to unexpected multi-threaded processing.

Analyze

Return after call of SetCommonProperty (setting common properties mt and memuse to members of CCommonMethodProps avoid that properties gets added regularly, so many handlers setting that via SetCoderProperties wouldn't consider that option at all, because option doesn't exists in the props.
For example this branch was previously never reachable:

7-Zip-zstd/CPP/7zip/Compress/ZstdEncoder.cpp

Line 58 in 8f52579

case NCoderPropID::kNumThreads:

Alternatively one'd rewrite it in every affected handler like in:

7-Zip-zstd/CPP/7zip/Archive/ZstdHandler.cpp

Line 276 in 8f52579

static HRESULT UpdateArchive(

with:

 static HRESULT UpdateArchive(
     UInt64 unpackSize,
     ISequentialOutStream *outStream,
-    const CProps &props,
+    const CSingleMethodProps &props,
 ...
   RINOK(props.SetCoderProps(encoderSpec, NULL));
+  if (props._numThreads_WasForced) {
+    encoderSpec->SetNumberOfThreads(props._numThreads);
+  }

Just it would be not quite backwards compatible (if one remove dup processing in SetCoderProperties at the same time).
As well as one should extend encoder and/or decoder of every handler, and this fix is minimalistic in opposite.

[mcmilk]

Thank you.

[mcmilk]

Good catch