Done. SimpleFeatureHolder wraps it's features() iterator using ListUtils.nonRemoveIterator(). All in cvs. Matthew ps Developers may wish to use this wrapper method in other places >>So, on balance, I guess we should throw NotImplementedException on >>remove(). Unless somebody has a bright idea?