[MERGE] Deprecate export-related InventoryEntry methods
Andrew Bennetts
andrew at canonical.com
Mon Jul 28 06:54:32 BST 2008
Martin Pool wrote:
> On Mon, Jul 28, 2008 at 12:58 PM, Andrew Bennetts <andrew at canonical.com> wrote:
[...]
> > It seems like a small shame to trade several small methods a function with
> > large switch statement, though. Perhaps we should refactor again so that we
[...]
> It seems like a case of double-dispatch based on both the entry kind
> and the destination format. This is much more closely coupled to the
> exporter than to the inventory kind, and it's probably more likely
> people will want to add new exporters, so it's probably happier there.
Yeah. I realise my suggestion wasn't very clear; I actually meant a registry
per exporter, for exactly this reason. There might be better ways to slice it,
of course.
In general pushing this functionality into the individual exporters and out of
InventoryEntry does make sense, so Ian's change is definitely heading in the
right direction.
-Andrew.
More information about the bazaar
mailing list