Daniel Norberg

Results 42 comments of 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...