Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
[imp-dev] API changes to TypeFactory?

Hi Jurgen,

The changes you made look reasonable in principle (separating out
the type dictionary from the TypeFactory), but I think these kind of
changes should be discussed on the mailing list first.

In particular, I see that the AnalysisManager is now broken - had I
not happened to look at the code while synching with the repo, I
wouldn't know about the  bogus use of "new TypeStore()" (marked
by a TODO), which, although it compiles, would silently fail to find
types mentioned in an analysis engine's type descriptors (I think).

That sort of silent breakage is confusing and could be hard to
diagnose...

Cheers,
 - Bob



Back to the top