Tag with quality and sub-category

2025-12-17 01:03:14 +01:00 · 2025-06-17 11:51:57 +01:00
parent 2e5d52e7e8
commit 97dceb3a1c
79 changed files with 204 additions and 110 deletions
--- a/java/ql/src/Advisory/Declarations/MissingOverrideAnnotation.ql
+++ b/java/ql/src/Advisory/Declarations/MissingOverrideAnnotation.ql
@@ -6,7 +6,9 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/missing-override-annotation
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
 */

 import java
--- a/Code/AvoidDeprecatedCallableAccess.ql
+++ b/Code/AvoidDeprecatedCallableAccess.ql
@@ -6,7 +6,9 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/deprecated-call
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
 *       non-attributable
 *       external/cwe/cwe-477
 */
--- a/java/ql/src/Advisory/Documentation/ImpossibleJavadocThrows.ql
+++ b/java/ql/src/Advisory/Documentation/ImpossibleJavadocThrows.ql
@@ -6,7 +6,9 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/inconsistent-javadoc-throws
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
 */

 import java
--- a/java/ql/src/Advisory/Documentation/SpuriousJavadocParam.ql
+++ b/java/ql/src/Advisory/Documentation/SpuriousJavadocParam.ql
@@ -6,7 +6,9 @@
 * @problem.severity recommendation
 * @precision very-high
 * @id java/unknown-javadoc-parameter
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
 */

 import java
--- a/java/ql/src/Compatibility/JDK9/JdkInternalAccess.ql
+++ b/java/ql/src/Compatibility/JDK9/JdkInternalAccess.ql
@@ -6,7 +6,9 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/jdk-internal-api-access
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
 */

 import java
--- a/java/ql/src/Compatibility/JDK9/UnderscoreIdentifier.ql
+++ b/java/ql/src/Compatibility/JDK9/UnderscoreIdentifier.ql
@@ -6,7 +6,9 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/underscore-identifier
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
 */

 import java
--- a/java/ql/src/DeadCode/UselessParameter.ql
+++ b/java/ql/src/DeadCode/UselessParameter.ql
@@ -5,7 +5,8 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/unused-parameter
- * @tags maintainability
+ * @tags quality
+ *       maintainability
 *       useless-code
 *       external/cwe/cwe-561
 */
--- a/Abuse/ChainedInstanceof.ql
+++ b/Abuse/ChainedInstanceof.ql
@@ -5,7 +5,9 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/chained-type-tests
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       complexity
 *       language-features
 */

--- a/Abuse/IterableIterator.ql
+++ b/Abuse/IterableIterator.ql
@@ -7,8 +7,9 @@
 * @problem.severity warning
 * @precision very-high
 * @id java/iterator-implements-iterable
- * @tags correctness
+ * @tags quality
 *       reliability
+ *       correctness
 */

 import java
--- a/Abuse/UselessNullCheck.ql
+++ b/Abuse/UselessNullCheck.ql
@@ -6,7 +6,8 @@
 * @problem.severity warning
 * @precision very-high
 * @id java/useless-null-check
- * @tags maintainability
+ * @tags quality
+ *       maintainability
 *       useless-code
 *       external/cwe/cwe-561
 */
--- a/Abuse/UselessTypeTest.ql
+++ b/Abuse/UselessTypeTest.ql
@@ -5,7 +5,9 @@
 * @problem.severity warning
 * @precision very-high
 * @id java/useless-type-test
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       useless-code
 *       language-features
 *       external/cwe/cwe-561
 */
--- a/Abuse/WrappedIterator.ql
+++ b/Abuse/WrappedIterator.ql
@@ -6,8 +6,9 @@
 * @problem.severity warning
 * @precision very-high
 * @id java/iterable-wraps-iterator
- * @tags correctness
+ * @tags quality
 *       reliability
+ *       correctness
 */

 import java
--- a/Bugs/Arithmetic/ConstantExpAppearsNonConstant.ql
+++ b/Bugs/Arithmetic/ConstantExpAppearsNonConstant.ql
@@ -5,8 +5,9 @@
 * @problem.severity warning
 * @precision very-high
 * @id java/evaluation-to-constant
- * @tags maintainability
- *       useless-code
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Bugs/Arithmetic/IntMultToLong.ql
+++ b/Bugs/Arithmetic/IntMultToLong.ql
@@ -6,10 +6,10 @@
 * @problem.severity warning
 * @precision very-high
 * @id java/integer-multiplication-cast-to-long
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 *       types
- *       quality
 *       external/cwe/cwe-190
 *       external/cwe/cwe-192
 *       external/cwe/cwe-197
--- a/Bugs/Arithmetic/LShiftLargerThanTypeWidth.ql
+++ b/Bugs/Arithmetic/LShiftLargerThanTypeWidth.ql
@@ -5,7 +5,9 @@
 * @problem.severity warning
 * @precision very-high
 * @id java/lshift-larger-than-type-width
- * @tags correctness
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Bugs/Arithmetic/WhitespaceContradictsPrecedence.ql
+++ b/Bugs/Arithmetic/WhitespaceContradictsPrecedence.ql
@@ -6,8 +6,9 @@
 * @problem.severity warning
 * @precision very-high
 * @id java/whitespace-contradicts-precedence
- * @tags maintainability
- *       readability
+ * @tags quality
+ *       reliability
+ *       correctness
 *       external/cwe/cwe-783
 */

--- a/Bugs/Collections/ArrayIndexOutOfBounds.ql
+++ b/Bugs/Collections/ArrayIndexOutOfBounds.ql
@@ -6,7 +6,8 @@
 * @problem.severity error
 * @precision high
 * @id java/index-out-of-bounds
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 *       exceptions
 *       external/cwe/cwe-193
--- a/Bugs/Collections/ContainsTypeMismatch.ql
+++ b/Bugs/Collections/ContainsTypeMismatch.ql
@@ -7,7 +7,8 @@
 * @problem.severity error
 * @precision very-high
 * @id java/type-mismatch-access
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 *       logic
 */
--- a/Bugs/Collections/ReadOnlyContainer.ql
+++ b/Bugs/Collections/ReadOnlyContainer.ql
@@ -5,9 +5,9 @@
 * @problem.severity error
 * @precision very-high
 * @id java/empty-container
- * @tags reliability
- *       maintainability
- *       useless-code
+ * @tags quality
+ *       reliability
+ *       correctness
 *       external/cwe/cwe-561
 */

--- a/Bugs/Collections/RemoveTypeMismatch.ql
+++ b/Bugs/Collections/RemoveTypeMismatch.ql
@@ -7,7 +7,8 @@
 * @problem.severity error
 * @precision very-high
 * @id java/type-mismatch-modification
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 *       logic
 */
--- a/Bugs/Collections/WriteOnlyContainer.ql
+++ b/Bugs/Collections/WriteOnlyContainer.ql
@@ -5,9 +5,9 @@
 * @problem.severity error
 * @precision very-high
 * @id java/unused-container
- * @tags maintainability
+ * @tags quality
+ *       maintainability
 *       useless-code
- *       quality
 *       external/cwe/cwe-561
 */

--- a/Bugs/Comparison/CompareIdenticalValues.ql
+++ b/Bugs/Comparison/CompareIdenticalValues.ql
@@ -6,7 +6,8 @@
 * @problem.severity error
 * @precision very-high
 * @id java/comparison-of-identical-expressions
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 *       logic
 */
--- a/Bugs/Comparison/EqualsArray.ql
+++ b/Bugs/Comparison/EqualsArray.ql
@@ -6,7 +6,8 @@
 * @problem.severity error
 * @precision very-high
 * @id java/equals-on-arrays
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 */

--- a/Bugs/Comparison/HashedButNoHash.ql
+++ b/Bugs/Comparison/HashedButNoHash.ql
@@ -6,7 +6,8 @@
 * @problem.severity error
 * @precision very-high
 * @id java/hashing-without-hashcode
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 */

--- a/Bugs/Comparison/IncomparableEquals.ql
+++ b/Bugs/Comparison/IncomparableEquals.ql
@@ -6,9 +6,10 @@
 * @problem.severity error
 * @precision very-high
 * @id java/equals-on-unrelated-types
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
- *       quality
+ *       external/cwe/cwe-571
 */

 import java
--- a/Bugs/Comparison/InconsistentEqualsHashCode.ql
+++ b/Bugs/Comparison/InconsistentEqualsHashCode.ql
@@ -6,9 +6,9 @@
 * @problem.severity error
 * @precision very-high
 * @id java/inconsistent-equals-and-hashcode
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
- *       quality
 *       external/cwe/cwe-581
 */

--- a/Bugs/Comparison/MissingInstanceofInEquals.ql
+++ b/Bugs/Comparison/MissingInstanceofInEquals.ql
@@ -6,9 +6,9 @@
 * @problem.severity error
 * @precision high
 * @id java/unchecked-cast-in-equals
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
- *       quality
 */

 import java
--- a/Bugs/Comparison/RefEqBoxed.ql
+++ b/Bugs/Comparison/RefEqBoxed.ql
@@ -6,9 +6,9 @@
 * @problem.severity error
 * @precision very-high
 * @id java/reference-equality-of-boxed-types
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
- *       quality
 *       external/cwe/cwe-595
 */

--- a/Bugs/Comparison/UselessComparisonTest.ql
+++ b/Bugs/Comparison/UselessComparisonTest.ql
@@ -7,7 +7,9 @@
 * @problem.severity warning
 * @precision very-high
 * @id java/constant-comparison
- * @tags correctness
+ * @tags quality
+ *       reliability
+ *       correctness
 *       logic
 *       external/cwe/cwe-570
 *       external/cwe/cwe-571
--- a/Bugs/Comparison/WrongNanComparison.ql
+++ b/Bugs/Comparison/WrongNanComparison.ql
@@ -6,7 +6,9 @@
 * @problem.severity error
 * @precision very-high
 * @id java/comparison-with-nan
- * @tags correctness
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Bugs/Concurrency/CallsToRunnableRun.ql
+++ b/Bugs/Concurrency/CallsToRunnableRun.ql
@@ -6,8 +6,8 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/call-to-thread-run
- * @tags reliability
- *       correctness
+ * @tags quality
+ *       reliability
 *       concurrency
 *       external/cwe/cwe-572
 */
--- a/Bugs/Concurrency/DoubleCheckedLocking.ql
+++ b/Bugs/Concurrency/DoubleCheckedLocking.ql
@@ -6,8 +6,8 @@
 * @problem.severity error
 * @precision high
 * @id java/unsafe-double-checked-locking
- * @tags reliability
- *       correctness
+ * @tags quality
+ *       reliability
 *       concurrency
 *       external/cwe/cwe-609
 */
--- a/Bugs/Concurrency/DoubleCheckedLockingWithInitRace.ql
+++ b/Bugs/Concurrency/DoubleCheckedLockingWithInitRace.ql
@@ -8,8 +8,8 @@
 * @problem.severity warning
 * @precision high
 * @id java/unsafe-double-checked-locking-init-order
- * @tags reliability
- *       correctness
+ * @tags quality
+ *       reliability
 *       concurrency
 *       external/cwe/cwe-609
 */
--- a/Bugs/Concurrency/NonSynchronizedOverride.ql
+++ b/Bugs/Concurrency/NonSynchronizedOverride.ql
@@ -6,8 +6,8 @@
 * @problem.severity warning
 * @precision very-high
 * @id java/non-sync-override
- * @tags reliability
- *       correctness
+ * @tags quality
+ *       reliability
 *       concurrency
 *       language-features
 *       external/cwe/cwe-820
--- a/Bugs/Concurrency/SynchOnBoxedType.ql
+++ b/Bugs/Concurrency/SynchOnBoxedType.ql
@@ -7,8 +7,8 @@
 * @problem.severity error
 * @precision very-high
 * @id java/sync-on-boxed-types
- * @tags reliability
- *       correctness
+ * @tags quality
+ *       reliability
 *       concurrency
 *       language-features
 *       external/cwe/cwe-662
--- a/Bugs/Concurrency/SynchSetUnsynchGet.ql
+++ b/Bugs/Concurrency/SynchSetUnsynchGet.ql
@@ -7,8 +7,8 @@
 * @problem.severity error
 * @precision very-high
 * @id java/unsynchronized-getter
- * @tags reliability
- *       correctness
+ * @tags quality
+ *       reliability
 *       concurrency
 *       language-features
 *       external/cwe/cwe-413
--- a/Bugs/Inheritance/NoNonFinalInConstructor.ql
+++ b/Bugs/Inheritance/NoNonFinalInConstructor.ql
@@ -6,7 +6,8 @@
 * @problem.severity error
 * @precision very-high
 * @id java/non-final-call-in-constructor
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 *       logic
 */
--- a/Typos/ContainerSizeCmpZero.ql
+++ b/Typos/ContainerSizeCmpZero.ql
@@ -5,7 +5,8 @@
 * @problem.severity warning
 * @precision very-high
 * @id java/test-for-negative-container-size
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 *       logic
 */
--- a/Typos/ContradictoryTypeChecks.ql
+++ b/Typos/ContradictoryTypeChecks.ql
@@ -7,9 +7,10 @@
 * @problem.severity error
 * @precision very-high
 * @id java/contradictory-type-checks
- * @tags correctness
+ * @tags quality
+ *       reliability
+ *       correctness
 *       logic
- *       quality
 */

 import java
--- a/Typos/MissingFormatArg.ql
+++ b/Typos/MissingFormatArg.ql
@@ -6,7 +6,9 @@
 * @problem.severity error
 * @precision very-high
 * @id java/missing-format-argument
- * @tags correctness
+ * @tags quality
+ *       reliability
+ *       correctness
 *       external/cwe/cwe-685
 */

--- a/Typos/MissingSpaceTypo.ql
+++ b/Typos/MissingSpaceTypo.ql
@@ -7,7 +7,9 @@
 * @problem.severity recommendation
 * @precision very-high
 * @id java/missing-space-in-concatenation
- * @tags readability
+ * @tags quality
+ *       maintainability
+ *       readability
 */

 import java
--- a/Typos/SelfAssignment.ql
+++ b/Typos/SelfAssignment.ql
@@ -5,7 +5,8 @@
 * @problem.severity error
 * @precision very-high
 * @id java/redundant-assignment
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 *       logic
 */
--- a/Typos/StringBufferCharInit.ql
+++ b/Typos/StringBufferCharInit.ql
@@ -6,8 +6,9 @@
 * @problem.severity error
 * @precision very-high
 * @id java/string-buffer-char-init
- * @tags reliability
- *       maintainability
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Typos/SuspiciousDateFormat.ql
+++ b/Typos/SuspiciousDateFormat.ql
@@ -5,8 +5,9 @@
 * @problem.severity warning
 * @precision high
 * @id java/suspicious-date-format
- * @tags correctness
- *       quality
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Typos/UnusedFormatArg.ql
+++ b/Typos/UnusedFormatArg.ql
@@ -7,7 +7,8 @@
 * @problem.severity warning
 * @precision very-high
 * @id java/unused-format-argument
- * @tags maintainability
+ * @tags quality
+ *       maintainability
 *       useless-code
 *       external/cwe/cwe-685
 */
--- a/Bugs/Nullness/NullAlways.ql
+++ b/Bugs/Nullness/NullAlways.ql
@@ -5,7 +5,8 @@
 * @problem.severity error
 * @precision very-high
 * @id java/dereferenced-value-is-always-null
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 *       exceptions
 *       external/cwe/cwe-476
--- a/Bugs/Nullness/NullExprDeref.ql
+++ b/Bugs/Nullness/NullExprDeref.ql
@@ -6,7 +6,8 @@
 * @problem.severity warning
 * @precision high
 * @id java/dereferenced-expr-may-be-null
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 *       exceptions
 *       external/cwe/cwe-476
--- a/Bugs/Nullness/NullMaybe.ql
+++ b/Bugs/Nullness/NullMaybe.ql
@@ -6,7 +6,8 @@
 * @problem.severity warning
 * @precision high
 * @id java/dereferenced-value-may-be-null
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 *       exceptions
 *       external/cwe/cwe-476
--- a/Leaks/CloseReader.ql
+++ b/Leaks/CloseReader.ql
@@ -6,10 +6,11 @@
 * @problem.severity warning
 * @precision high
 * @id java/input-resource-leak
- * @tags efficiency
- *       correctness
+ * @tags quality
+ *       reliability
+ *       performance
+ *       efficiency
 *       resources
- *       quality
 *       external/cwe/cwe-404
 *       external/cwe/cwe-772
 */
--- a/java/ql/src/Likely
+++ b/java/ql/src/Likely
@@ -5,7 +5,9 @@
 * @problem.severity warning
 * @precision high
 * @id java/database-resource-leak
- * @tags correctness
+ * @tags quality
+ *       reliability
+ *       performance
 *       resources
 *       external/cwe/cwe-404
 *       external/cwe/cwe-772
--- a/Leaks/CloseWriter.ql
+++ b/Leaks/CloseWriter.ql
@@ -6,10 +6,11 @@
 * @problem.severity warning
 * @precision high
 * @id java/output-resource-leak
- * @tags efficiency
- *       correctness
+ * @tags quality
+ *       reliability
+ *       performance
+ *       efficiency
 *       resources
- *       quality
 *       external/cwe/cwe-404
 *       external/cwe/cwe-772
 */
--- a/Bugs/Statements/ContinueInFalseLoop.ql
+++ b/Bugs/Statements/ContinueInFalseLoop.ql
@@ -8,7 +8,9 @@
 * @id java/continue-in-false-loop
 * @problem.severity warning
 * @precision high
- * @tags correctness
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Bugs/Statements/PartiallyMaskedCatch.ql
+++ b/Bugs/Statements/PartiallyMaskedCatch.ql
@@ -6,7 +6,8 @@
 * @problem.severity warning
 * @precision high
 * @id java/unreachable-catch-clause
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 *       exceptions
 *       external/cwe/cwe-561
--- a/Bugs/Statements/UseBraces.ql
+++ b/Bugs/Statements/UseBraces.ql
@@ -6,7 +6,9 @@
 * @problem.severity warning
 * @precision very-high
 * @id java/misleading-indentation
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
 *       correctness
 *       logic
 */
--- a/Bugs/Termination/ConstantLoopCondition.ql
+++ b/Bugs/Termination/ConstantLoopCondition.ql
@@ -7,7 +7,9 @@
 * @problem.severity warning
 * @precision very-high
 * @id java/constant-loop-condition
- * @tags correctness
+ * @tags quality
+ *       reliability
+ *       correctness
 *       external/cwe/cwe-835
 */

--- a/java/ql/src/Performance/InefficientEmptyStringTest.ql
+++ b/java/ql/src/Performance/InefficientEmptyStringTest.ql
@@ -5,8 +5,9 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/inefficient-empty-string-test
- * @tags efficiency
- *       maintainability
+ * @tags quality
+ *       reliability
+ *       performance
 */

 import java
--- a/java/ql/src/Performance/InefficientKeySetIterator.ql
+++ b/java/ql/src/Performance/InefficientKeySetIterator.ql
@@ -5,8 +5,9 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/inefficient-key-set-iterator
- * @tags efficiency
- *       maintainability
+ * @tags quality
+ *       reliability
+ *       performance
 */

 import java
--- a/java/ql/src/Performance/InefficientOutputStream.ql
+++ b/java/ql/src/Performance/InefficientOutputStream.ql
@@ -6,7 +6,10 @@
 * @problem.severity warning
 * @precision very-high
 * @id java/inefficient-output-stream
- * @tags efficiency
+ * @tags quality
+ *       reliability
+ *       performance
+ *       efficiency
 */

 import java
--- a/java/ql/src/Performance/InefficientPrimConstructor.ql
+++ b/java/ql/src/Performance/InefficientPrimConstructor.ql
@@ -5,8 +5,10 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/inefficient-boxed-constructor
- * @tags efficiency
- *       maintainability
+ * @tags quality
+ *       reliability
+ *       performance
+ *       efficiency
 */

 import java
--- a/java/ql/src/Performance/InnerClassCouldBeStatic.ql
+++ b/java/ql/src/Performance/InnerClassCouldBeStatic.ql
@@ -6,8 +6,10 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/non-static-nested-class
- * @tags efficiency
- *       maintainability
+ * @tags quality
+ *       reliability
+ *       performance
+ *       efficiency
 */

 import java
--- a/java/ql/src/Performance/NewStringString.ql
+++ b/java/ql/src/Performance/NewStringString.ql
@@ -6,8 +6,10 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/inefficient-string-constructor
- * @tags efficiency
- *       maintainability
+ * @tags quality
+ *       reliability
+ *       performance
+ *       efficiency
 */

 import java
--- a/Types/BoxedVariable.ql
+++ b/Types/BoxedVariable.ql
@@ -6,7 +6,9 @@
 * @problem.severity warning
 * @precision very-high
 * @id java/non-null-boxed-variable
- * @tags readability
+ * @tags quality
+ *       maintainability
+ *       readability
 *       types
 */

--- a/java/ql/src/Violations
+++ b/java/ql/src/Violations
@@ -6,7 +6,8 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/unused-reference-type
- * @tags maintainability
+ * @tags quality
+ *       maintainability
 *       useless-code
 *       external/cwe/cwe-561
 */
--- a/Code/InterfaceCannotBeImplemented.ql
+++ b/Code/InterfaceCannotBeImplemented.ql
@@ -6,7 +6,8 @@
 * @problem.severity warning
 * @precision very-high
 * @id java/unimplementable-interface
- * @tags maintainability
+ * @tags quality
+ *       maintainability
 *       useless-code
 */

--- a/java/ql/src/Violations
+++ b/java/ql/src/Violations
@@ -5,7 +5,8 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/local-variable-is-never-read
- * @tags maintainability
+ * @tags quality
+ *       maintainability
 *       useless-code
 *       external/cwe/cwe-561
 */
--- a/java/ql/src/Violations
+++ b/java/ql/src/Violations
@@ -6,7 +6,8 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/unused-label
- * @tags maintainability
+ * @tags quality
+ *       maintainability
 *       useless-code
 *       external/cwe/cwe-561
 */
--- a/Practice/Declarations/NoConstantsOnly.ql
+++ b/Practice/Declarations/NoConstantsOnly.ql
@@ -6,7 +6,9 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/constants-only-interface
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
 *       modularity
 */

--- a/Handling/IgnoreExceptionalReturn.ql
+++ b/Handling/IgnoreExceptionalReturn.ql
@@ -6,8 +6,9 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/ignored-error-status-of-call
- * @tags reliability
- *       correctness
+ * @tags quality
+ *       reliability
+ *       error-handling
 *       external/cwe/cwe-391
 */

--- a/Handling/NumberFormatException.ql
+++ b/Handling/NumberFormatException.ql
@@ -6,7 +6,9 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/uncaught-number-format-exception
- * @tags reliability
+ * @tags quality
+ *       reliability
+ *       error-handling
 *       external/cwe/cwe-248
 */

--- a/Hiding/AbstractToConcreteCollection.ql
+++ b/Hiding/AbstractToConcreteCollection.ql
@@ -6,8 +6,9 @@
 * @problem.severity warning
 * @precision very-high
 * @id java/abstract-to-concrete-cast
- * @tags reliability
+ * @tags quality
 *       maintainability
+ *       complexity
 *       modularity
 *       external/cwe/cwe-485
 */
--- a/Hiding/ExposeRepresentation.ql
+++ b/Hiding/ExposeRepresentation.ql
@@ -6,8 +6,9 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/internal-representation-exposure
- * @tags reliability
- *       maintainability
+ * @tags quality
+ *       reliability
+ *       correctness
 *       modularity
 *       external/cwe/cwe-485
 */
--- a/Conventions/AmbiguousOuterSuper.ql
+++ b/Conventions/AmbiguousOuterSuper.ql
@@ -6,7 +6,8 @@
 * @problem.severity warning
 * @precision very-high
 * @id java/subtle-inherited-call
- * @tags reliability
+ * @tags quality
+ *       maintainability
 *       readability
 */

--- a/Conventions/ConfusingMethodNames.ql
+++ b/Conventions/ConfusingMethodNames.ql
@@ -6,7 +6,8 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/confusing-method-name
- * @tags maintainability
+ * @tags quality
+ *       maintainability
 *       readability
 *       naming
 */
--- a/Conventions/ConfusingOverloading.ql
+++ b/Conventions/ConfusingOverloading.ql
@@ -7,7 +7,8 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/confusing-method-signature
- * @tags maintainability
+ * @tags quality
+ *       maintainability
 *       readability
 *       naming
 */
--- a/Conventions/LocalShadowsFieldConfusing.ql
+++ b/Conventions/LocalShadowsFieldConfusing.ql
@@ -6,7 +6,8 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/local-shadows-field
- * @tags maintainability
+ * @tags quality
+ *       maintainability
 *       readability
 */

--- a/Conventions/SameNameAsSuper.ql
+++ b/Conventions/SameNameAsSuper.ql
@@ -5,7 +5,8 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/class-name-matches-super-class
- * @tags maintainability
+ * @tags quality
+ *       maintainability
 *       readability
 *       naming
 */
--- a/Calls/CallsToStringToString.ql
+++ b/Calls/CallsToStringToString.ql
@@ -5,7 +5,9 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/useless-tostring-call
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       useless-code
 */

 import java
--- a/Calls/DefaultToString.ql
+++ b/Calls/DefaultToString.ql
@@ -6,8 +6,9 @@
 * @problem.severity recommendation
 * @precision high
 * @id java/call-to-object-tostring
- * @tags reliability
- *       maintainability
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/java/ql/src/Violations
+++ b/java/ql/src/Violations
@@ -6,7 +6,9 @@
 * @problem.severity recommendation
 * @precision very-high
 * @id java/print-array
- * @tags maintainability
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java