This inner type isn't necessary, is it?

Why can't the method be:

  if (root != null)
    return root.iterator(new MutableObjectId(), reader);
  else:
    return Collections.<Note>emptyList().iterator();