CombinedGraphIndex API changes broken plugins

[Robert Collins]

The change adding index_names to CombinedGraphIndex broke some plugins that were using it by mutating the list they passed in - it naively zips the two lists, when some names are missing indices are dropped. I think this should be guarded against.

[John A Meinel (jameinel)]

I believe the most reasonable was to have it check that the len() of appropriate lists matched, so we could at least do some change checking.

[Jelmer Vernooij (jelmer)]

Hi Rob,

Was this bzr-search specifically? Since this was so long ago (and not on the list to be fixed for 2.3), do you still think this is something that should be fixed?

[Launchpad Janitor (janitor)]

[Expired for Bazaar because there has been no activity for 60 days.]