Refer to "helpful NullPointerException" message for throw-null-literal.ql

Marcono1234 · Marcono1234 · commit 0f73bcb2c6e9 · 2023-05-28T17:16:10.000+02:00
diff --git a/codeql-custom-queries-java/queries/api-misuse/throw-null-literal.ql b/codeql-custom-queries-java/queries/api-misuse/throw-null-literal.ql
@@ -5,6 +5,10 @@
  * For clarity it would be better to explicitly create a `NullPointerException`
  * and possibly provide a meaningful message to its constructor.
  * 
+ * The "helpful `NullPointerException`" feature ([JEP-358](https://openjdk.org/jeps/358))
+ * makes it even clearer that `throw null` is not the intended usage:
+ * > NullPointerException: Cannot throw exception because "null" is null
+ * 
  * See also Error Prone pattern [ThrowNull](https://errorprone.info/bugpattern/ThrowNull).
  * 
  * @kind problem

Original file line number	Diff line number	Diff line change
`@@ -5,6 +5,10 @@`
`5`	`5`	* For clarity it would be better to explicitly create a `NullPointerException`
`6`	`6`	`* and possibly provide a meaningful message to its constructor.`
`7`	`7`	`*`
	`8`	+ * The "helpful `NullPointerException`" feature ([JEP-358](https://openjdk.org/jeps/358))
	`9`	+ * makes it even clearer that `throw null` is not the intended usage:
	`10`	`+ * > NullPointerException: Cannot throw exception because "null" is null`
	`11`	`+ *`
`8`	`12`	`* See also Error Prone pattern [ThrowNull](https://errorprone.info/bugpattern/ThrowNull).`
`9`	`13`	`*`
`10`	`14`	`* @kind problem`