Surface::makeImageSnapshot is correctly marked as @Nullable because _nMakeImageSnapshotR can return a null pointer. However, it doesn't check whether _nMakeImageSnapshotR returns null and instead always calls the Image constructor which will crash if the pointer is null.
tonsky
commented
2 years ago
Surface::makeImageSnapshotis correctly marked as@Nullablebecause_nMakeImageSnapshotRcan return a null pointer. However, it doesn't check whether_nMakeImageSnapshotRreturns null and instead always calls theImageconstructor which will crash if the pointer is null.