Merge pull request #1192 from geoffw0/severity

CPP: Change some query severities

Author: jbj

cpp/ql/src/Best Practices/SloppyGlobal.ql

@@ -2,8 +2,8 @@
  * @name Short global name
  * @description Global variables should have descriptive names, to help document their use, avoid namespace pollution and reduce the risk of shadowing with local variables.
  * @kind problem
- * @problem.severity warning
- * @precision high
+ * @problem.severity recommendation
+ * @precision very-high
  * @id cpp/short-global-name
  * @tags maintainability
  */

cpp/ql/src/jsf/4.13 Functions/AV Rule 107.ql

@@ -2,8 +2,8 @@
  * @name Function declared in block
  * @description Functions should always be declared at file scope. It is confusing to declare a function at block scope, and the visibility of the function is not what would be expected.
  * @kind problem
- * @problem.severity warning
- * @precision high
+ * @problem.severity recommendation
+ * @precision very-high
  * @id cpp/function-in-block
  * @tags maintainability
  *       readability