Updated C33010

Updated to new format

Author: MugBergerFries

docs/code-quality/c33010.md

@@ -4,25 +4,27 @@ description: C33010 warning for enums
 keywords: c33010
 author: hwisungi
 ms.author: hwisungi
-ms.date: 06/20/2020
+ms.date: 09/08/2022
 ms.topic: reference
-f1_keywords: ["C33010"]
+f1_keywords: ["C33010", "UNCHECKED_LOWER_BOUND_FOR_ENUMINDEX", "__WARNING_UNCHECKED_LOWER_BOUND_FOR_ENUMINDEX"]
 helpviewer_keywords: ["C33010"]
 dev_langs: ["C++"]
 ---
-# C33010
+# Warning C33010
 
-> Warning C33010: Unchecked lower bound for enum 'enum' used as index.
+> Unchecked lower bound for enum '\**enum_name*' used as index.
 
-This warning is triggered for an enum that is used as an index into an array,
-if the upper bound is checked for its value, but not the lower bound.
+This warning is triggered if an enum is both used as an index into an array and isn't checked on the lower bound.
+
+## Remarks
+
+Code using enumerated types as indexes for arrays will often check for the upper bound in order to ensure the index is not out of range. Because an enum variable is signed by default, it can have a negative value. If it is used as an index into an array of values or an array of function pointers, a negative value can allow arbitrary memory to be read, used, or even executed.
+
+Code analysis name: UNCHECKED_LOWER_BOUND_FOR_ENUMINDEX
 
 ## Example
 
-Code using enumerated types as indexes for arrays will often check for the upper bound
-in order to ensure the index is not out of range. Because an enum variable is signed by default,
-it can have a negative value. If it is used as an index into an array of values or an array of function pointers,
-a negative value can allow arbitrary memory to be read, used, or even executed.
+The following code generates this warning. `idx` is used as an index to access `functions`, but the lower bound is never checked.
 
 ```cpp
 typedef void (*PFN)();
@@ -41,14 +43,13 @@ void foo(Index idx, PFN(&functions)[5])
     if (idx > Index::Max)
         return;
 
-    auto pfn = functions[static_cast<int>(idx)];    // C33010
+    auto pfn = functions[static_cast<int>(idx)];
     if (pfn != nullptr)
         (*pfn)();
-    // ......
 }
 ```
 
-These warnings are corrected by checking the index value for lower bound as well:
+These following code remediates this warning by checking the lower bound as well to ensure `idx` is not negative:
 
 ```cpp
 typedef void (*PFN)();
@@ -67,10 +68,9 @@ void foo(Index idx, PFN(&functions)[5])
     if (idx < Index::Zero || idx > Index::Max)
         return;
 
-    auto pfn = functions[static_cast<int>(idx)];    // OK
+    auto pfn = functions[static_cast<int>(idx)];
     if (pfn != nullptr)
         (*pfn)();
-    // ......
 }
 ```