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