Merge pull request #5735 from MicrosoftDocs/main

Albertyang0 · web-flow · commit b973448d2f2b · 2025-01-15T10:29:26.000-08:00
1/15/2025 AM Publish
diff --git a/docs/code-quality/c33001.md b/docs/code-quality/c33001.md
@@ -13,14 +13,23 @@ helpviewer_keywords: ["C33001"]
 
 ## Remarks
 
-This warning is triggered when an uninitialized `VARIANT` is passed to an API such as `VariantClear`
-that expects an initialized `VARIANT`.
+This warning is triggered when an uninitialized `VARIANT` is passed to an API, such as `VariantClear`, that clears the object. Initialize the `VARIANT` before passing it to these functions so it can be properly cleared.
+
+This warning applies to these functions:
+
+* `VariantClear`
+* `PropVariantClear`
+* `VariantCopy`
+* `VariantCopyInd`
+* `VariantChangeType`
+* `VariantChangeTypeEx`
+* `DestroyPropVariant`
 
 Code analysis name: `VARIANTCLEAR_UNINITIALIZED`
 
 ## Example
 
-The following sample code causes warning C33001:
+The following code causes warning C33001:
 
 ```cpp
 #include <Windows.h>
@@ -36,11 +45,11 @@ HRESULT foo(bool some_condition)
         //...
     }
 
-    VariantClear(&var);     // C33001
+    VariantClear(&var);  // C33001
 }
 ```
 
-These warnings are corrected by ensuring `VariantClear` is called only for a properly initialized `VARIANT`:
+In this example, the warning is corrected by calling `VariantClear` only after `var` has been initialized:
 
 ```cpp
 #include <Windows.h>
@@ -54,12 +63,12 @@ HRESULT foo(bool some_condition)
         //...
         VariantInit(&var);
         //...
-        VariantClear(&var);     // C33001
+        VariantClear(&var);  // OK
     }
 }
 ```
 
 ## See also
 
-[C33004](./c33004.md)
+[C33004](./c33004.md)\
 [C33005](./c33005.md)
