Open john-h-kastner-aws opened 8 months ago
Maybe ask Lucas to review?
@john-h-kastner-aws Raising the question, is this change planned for reconsideration at some point? It would be helpful to remove the 3 MB dependency on Google Guava for those integrating with the Cedar Java library.
Requires the following changes:
Use
copyOf
andunmodifiableSet
functions to replaceImmutableSet
. Prefer usingcopyOf
because it creates an unmodifiable copy of the set or list, so mutation to the input does not reflect in the new object.unmodifiableSet
can still be used to return an immutable view of a set without causing any copying.Roll our own memoizing supplier. This change should be reviewed by someone with more Java knowledge, but, AFAIK, the behavior should be the same.
Issue #, if available:
86
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.