wpilibsuite / native-utils

Gradle utility plugin for compiling native libraries
Other
4 stars 21 forks source link

Handle invalid vendordep JSON files better #228

Open rzblue opened 5 days ago

rzblue commented 5 days ago

Missing/incorrect keys usually results in a difficult to debug NPE when the value is accessed.

sciencewhiz commented 5 days ago

It looks like there's code to throw exceptions for many errors. Do you have examples of files that have NPEs?

https://github.com/wpilibsuite/native-utils/blob/d7a64d61551c9d915488b10b2d309968de1eb5a5/src/main/java/edu/wpi/first/nativeutils/vendordeps/WPIVendorDepsExtension.java#L199-L224