[Cdk-devel] IAtomContainer iterators

[Matt S. <mes...@gm...>]

I think it might be a good idea to rename the methods that return
iterators eg. atoms()  to something like atomIterator() and have
methods such as atoms() return an Iterable<IAtom>

public Iterable<IAtom> atoms()
{
    return new Iterable<IAtom>()
    {
         public Iterator<IAtom> iterator()
         {
               return atomIterator();
         }
    }
}

This would allow people to use foreach loops with like so

for(IAtom a : mol.atoms())

I think this is a little cleaner than looping using the iterator
directly. What do people think about this?

-Matt