search

brenoepics / morningstar

An open-source Arcturus Community Fork.
GNU General Public License v3.0
3 stars 2 forks source link

Update RoomLayout Null - [closed] #1530

Closed brenoepics closed 1 year ago

brenoepics commented 2 years ago

In GitLab by @xjoao on Mar 21, 2022, 08:01

Merges update-roomlayout -> dev

Argument 'this.findTile(openList, newTile.x, newTile.y)' might be null

brenoepics commented 2 years ago

In GitLab by @Remco on Mar 22, 2022, 09:11

You can't fix nullpointers with methods that throw nullpointers

public static <T> T requireNonNull(T obj) { if (obj == null) throw new NullPointerException(); return obj; }

Also: we really appreciate that you're trying to contribute but your contributions with like IntelliJ suggestions don't really add any value to Morningstar.