<Swing Dev> RFR: 8231041: Spotbugs: Null check of value previously dereferenced in java.desktop

Phil Race prr at openjdk.java.net
Tue Dec 22 20:20:57 UTC 2020


On Tue, 22 Dec 2020 19:39:57 GMT, Andrey Turbanov <github.com+741251+turbanoff at openjdk.org> wrote:

> I checked JDK16 repository in SpotBugs 4.2.0
> ![изображение](https://user-images.githubusercontent.com/741251/102926543-405c9000-44a6-11eb-9772-bf81a22ec78a.png)
> I fixed only places in java.desktop module.
> I didn't fixed places, where dereferencing is done inside method.

Did it occur that maybe the previous de-reference without a null check is the real problem ?
The proposed Raster change actually needs to be addressed as discussed inhttps://bugs.openjdk.java.net/browse/JDK-8255800 and so should not be part of this proposed change. DataBufferUShort may be the same.

But *all* of them need to be re-examined rather than just blindly updating them as some tool suggests.

-------------

Changes requested by prr (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/1869


More information about the swing-dev mailing list