Hi Steve,

> > - class AbstractMapReader should contain common code in (at least two)
> > readers
> > - class AbstractMapProcessor should contain common code in  (at least two)
> > processors
> > - class AbstractOSMWriter should contain common code in  (at least two)
> > writers
> 
> Yes, that sounds fine.

OK, the pbf reader part is a bit difficult, because the interfaces are in the 
jar file, but the rest 
looks good to me.

> 
> > b) The NodeCollector (used with parm --legacy-mode) seems to be obsolete to
> > me, I want to remove it
> > c) A few other helper classes are no longer in use (e.g. IntList, IntObjMap,
> > SparseInt2ShortMap). I'd like to move them to a folder called obsolete
> 
> Let just remove everything that is not used.

OK

> 
> Over time there has been three different versions of the program that work
> in substantially different ways, so there are a few obsolete things.
> 
> I'll set up a svn account for you too.

OK, please let me know how I can use it :-)

Ciao,
Gerd
                                          
_______________________________________________
mkgmap-dev mailing list
mkgmap-dev@lists.mkgmap.org.uk
http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev

Reply via email to