Daniel Norberg
Daniel Norberg
This indeed seems like a bug. Would you like to contribute a fix?
Sounds good. Don’t remember now exactly why spotbugs complained though.
~~@lepistone That indeed seems like a simpler approach. 👍~~
Would using `@SuppressWarnings("varargs")` help in this case though? Adding it to a method with a reifiable vararg parameter does not seem to make IntelliJ happier about the method. Javac with...
@lepistone Indeed. I had simply failed to enable `showWarnings`. https://github.com/danielnorberg/auto-matter/pull/72
Although the `@SuppressWarnings("varargs")` has been released in `0.15.3`, I guess it could still be neat to avoid both `@SafeVarargs` and `@SuppressWarnings` in code generated for reifiable types.
@bndbsh I attempted to handle nested types here, please take a look: https://github.com/mninja/auto-matter/pull/1
TODO: - Allow `Optional` fields? - Allow collection/map fields? - Add `@AutoMatter.Default` annotation instead of using `@AutoMatter.Field`?
**Note**: does not work well with gson as it bypasses the value constructor.
This is a much-needed refactoring. Kudos! Looks great overall, just a few small comments. * Eliminate circular imports and dependencies. * Everything should be package-private or private. * Leave everything...