Java: Ignore char array based closeables for CloseReader.ql and CloseWriter.ql

[Marcono1234]

Adds CharArrayReader and CharArrayWriter as safe closeables to reduce false positives.

[smowton]

Should note in the PR and/or a commit message: guessing you removed StringInputStream because that likely intended to refer to https://docs.aws.amazon.com/AWSJavaSDK/latest/javadoc/com/amazonaws/util/StringInputStream.html which is a subclass of ByteArrayInputStream in any case?

[Marcono1234]

Ah, sorry I have mentioned it in #5868 (comment), but have already resolved that.
Neither the Git history indicates what that class is supposed to match nor did the qhelp file mention it. And doing a quick web search did not yield any results; though it looks like I missed the AWS class.

If you want I can split the removal into a separate commit, or edit the commit message. Which one do you prefer?

[smowton]

The safe option would be to leave it alone, if we're unsure what it was supposed to do.

[intrigus-lgtm]

This still needs a change-note.
Something like

lgtm,codescanning
* The 'Potential input resource leak' (java/input-resource-leak) and
'Potential output resource leak' (java/output-resource-leak) queries have been improved to produce fewer false positives.

[Marcono1234]

The safe option would be to leave it alone, if we're unsure what it was supposed to do.

Added it back, but left a comment to explain why an unqualified name is used.

This still needs a change-note.
Something like

Thanks! I have used a slightly different wording, but I hope it is fine nonetheless.

[smowton]

Over to @aschackmull for final review

[smowton]

Oops, indeed US spelling is standard for this codebase