Remove all NullPointerExceptions

teragrep / cfe_31

0 stars 0 forks source link

Remove all NullPointerExceptions #51

Open MoonBow-1 opened 2 months ago

MoonBow-1 commented 2 months ago

Description

NullPointerExceptions shouldn't be thrown or caught, so refactoring is required for methods that throw them.

MoonBow-1 commented 2 months ago

getCaptureDefinitionWithSink() method in GenerateFiles is already refactored in #49, so I'm going to skip them

MoonBow-1 commented 1 month ago

@51-code Could you take a look at the CFE_31 code base on the remove-nullpointerexceptions branch, and just verify that there aren't any more NullPointer related code left?

So far:

Project doesn't contain any NullPointerExceptions
Nullable and NotNull annotations have been removed
Extra checks have been implemented for objects to verify their own content (valid() method)

There might be some things that need fixing, but I think those should be their own issues