Address review feedback

Author: knewbury01

cpp/misra/src/rules/RULE-10-1-2/VolatileQualifierNotUsedAppropriately.ql

@@ -1,8 +1,9 @@
 /**
  * @id cpp/misra/volatile-qualifier-not-used-appropriately
  * @name RULE-10-1-2: The volatile qualifier shall be used appropriately
- * @description Using the volatile qualifier on certain entities can lead to undefined behavior or
- *              code that is hard to understand.
+ * @description Using the volatile qualifier on certain entities has behavior that is not
+ *              well-defined and can make code harder to understand, especially as its application
+ *              to these entities does not prevent data races or guarantee safe multithreading.
  * @kind problem
  * @precision very-high
  * @problem.severity error
@@ -28,4 +29,4 @@ where
     d instanceof Function or
     d.(Variable).isStructuredBinding()
   )
-select d, "Volatile entity declared."
+select d, "Volatile entity '" + d.getName() + "' declared."

cpp/misra/test/rules/RULE-10-1-2/VolatileQualifierNotUsedAppropriately.expected

@@ -1,7 +1,7 @@
-| test.cpp:2:16:2:16 | g1 | Volatile entity declared. |
-| test.cpp:4:21:4:21 | p | Volatile entity declared. |
-| test.cpp:5:16:5:16 | x | Volatile entity declared. |
-| test.cpp:9:18:9:18 | a | Volatile entity declared. |
-| test.cpp:12:14:12:15 | f1 | Volatile entity declared. |
-| test.cpp:15:23:15:23 | p | Volatile entity declared. |
-| test.cpp:19:16:19:16 | m | Volatile entity declared. |
+| test.cpp:2:16:2:16 | g1 | Volatile entity 'g1' declared. |
+| test.cpp:5:21:5:21 | p | Volatile entity 'p' declared. |
+| test.cpp:6:16:6:16 | x | Volatile entity 'x' declared. |
+| test.cpp:10:18:10:18 | a | Volatile entity 'a' declared. |
+| test.cpp:13:14:13:15 | f1 | Volatile entity 'f1' declared. |
+| test.cpp:16:23:16:23 | p | Volatile entity 'p' declared. |
+| test.cpp:20:16:20:16 | m | Volatile entity 'm' declared. |

cpp/misra/test/rules/RULE-10-1-2/test.cpp

@@ -1,5 +1,6 @@
 int g[1] = {1};
 auto volatile [g1] = g; // NON_COMPLIANT
+volatile int g2;        // COMPLIANT
 
 void f(volatile int p) { // NON_COMPLIANT
   volatile int x = 1;    // NON_COMPLIANT

rule_packages/cpp/Declarations4.json

@@ -7,7 +7,7 @@
       },
       "queries": [
         {
-          "description": "Using the volatile qualifier on certain entities can lead to undefined behavior or code that is hard to understand.",
+          "description": "Using the volatile qualifier on certain entities has behavior that is not well-defined and can make code harder to understand, especially as its application to these entities does not prevent data races or guarantee safe multithreading.",
           "kind": "problem",
           "name": "The volatile qualifier shall be used appropriately",
           "precision": "very-high",