[libcontacts] Do not use iterators during coalescing

This practice is possibly affected by iterator invalidation in some
way I can't locate.  In any case, index-based operations are not
going to make any performance difference.
Merge pull request #58 from matthewvogt/remove-iterators

[libcontacts] Do not use iterators during coalescing