[tinyTiM-discuss] ITopicMapReader design

[Lars H. <he...@se...>]

Hi all,

The current (unused) ITopicMapReader interface provides three
different methods to deserialize topic maps.
<http://tinytim.svn.sourceforge.net/viewvc/tinytim/tinytim-mio/trunk/src/main/java/org/tinytim/mio/ITopicMapReader.java?view=markup>

I'd like to switch to minimised ITopicMapReader which provides just
a "void read() throws IOException" method. A nice side effect would be
that "read()" could do anything (read from a InputStream, read from a
database) etc.

The common TopicMapReader implementation would take an InputStream / a
File and a tmapi.core.TopicMap instance in the constructor or
alternatively provide some setters (even if I'd prefer immutable
implementations).

Even if the design change wouldn't break backwards compatibility
(since we never published the ITopicMapReader), I like to
hear your opinion.

What do you think? Any objections?

Best regards,
Lars
-- 
Semagia 
<http://www.semagia.com>