Hmm. I would personally trade the compile warning to avoid creating the
wrapper objects for the map and entries, and re-calculating the hashes, but
I defer.


On Mon, Dec 9, 2013 at 12:25 PM, ajs6f <notifications@github.com> wrote:

> In fcrepo-kernel/src/main/java/org/fcrepo/kernel/observer/FedoraEvent.java:
>
> > @@ -63,9 +80,14 @@ public String getIdentifier() throws RepositoryException {
> >          return e.getIdentifier();
> >      }
> >
> > +    @SuppressWarnings("unchecked")
>
> This is more verbose (below) but eliminates the suppression. (Suppress all
> suppressions!)
>
> @Override
> public Map<?, ?> getInfo() throws RepositoryException {
>     if (memoizedInfo == null) {
>         memoizedInfo = new HashMap<>(getRawInfo());
>         memoizedInfo.put(NODE_TYPE_KEY, this.nodeType);
>     }
>     return memoizedInfo;
> }
>
> public Map<?,?> getRawInfo() throws RepositoryException {
>     return e.getInfo();
> }
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/futures/fcrepo4/pull/197/files#r8204311>
> .
>