Hi Bernd,

the good news: I can reproduce the problems, the bad news:
I did not yet find a solution. Some are special cases introduced
by the new code, some caused problems before but mkgmap did
not report the problem.

Regarding your question if you should try to fix problems in OSM:
When mkgmap reports an "internal error" the problem may be caused
by wrong OSM data, but mkgmap  should also be fixed to handle the
situation. I prefer these messages compared to a simple
assertion which often doesn't report the file that contains the problem.
I also code such error messages when I expect a problem in
special cases but my test data did not yet show it.
If you change the OSM data, always make sure to post a link to the
file that allows to reproduce the problem.

Gerd


> Date: Sat, 25 Apr 2015 12:53:01 +0200
> From: weigelt.bernd@web.de
> To: mkgmap-dev@lists.mkgmap.org.uk
> Subject: Re: [mkgmap-dev] error message in Cologne
>
> I have read the comment on r3541, but housenumbers2 was on a really good way, so i want to give it a chance. I can build my maps with trunk in the next days
>
> Hier sollte eigentlich eine Signatur stehen.
>
>
>
> -----Original Message-----
> From: Gerd Petermann <gpetermann_muenchen@hotmail.com>
> To: "mkgmap-dev@lists.mkgmap.org.uk" <mkgmap-dev@lists.mkgmap.org.uk>
> Sent: Sa., 25 Apr. 2015 12:28
> Subject: Re: [mkgmap-dev] error message in Cologne
>
> Hi Bernd,
>
> thanks for reporting. Please note that the branch is
> very experimental since r3541. I'll try to fix the problems
> that you have reported, but you should not expect good
> results until I post that I think a test should be done.
>
> Gerd
>
> > From: weigelt.bernd@web.de
> > To: mkgmap-dev@lists.mkgmap.org.uk
> > Date: Sat, 25 Apr 2015 12:04:36 +0200
> > Subject: Re: [mkgmap-dev] error message in Cologne
> >
> > Am Samstag, 25. April 2015, 11:54:33 schrieb Bernd Weigelt:
> > > get lots of other problems with ...
> >
> > there is a second error
> > http://files.mkgmap.org.uk/download/263/65010116.o5m
> >
> > java.lang.ArrayIndexOutOfBoundsException: -2
> > at
> > uk.me.parabola.mkgmap.osmstyle.housenumber.ExtNumbers.splitLargeGaps(ExtNumbers.java:1600)
> > at
> > uk.me.parabola.mkgmap.osmstyle.housenumber.HousenumberRoad.improveSearchResults(HousenumberRoad.java:602)
> > at
> > uk.me.parabola.mkgmap.osmstyle.housenumber.HousenumberGenerator.generate(HousenumberGenerator.java:682)
> > at
> > uk.me.parabola.mkgmap.osmstyle.StyledConverter.end(StyledConverter.java:607)
> > at
> > uk.me.parabola.mkgmap.reader.osm.ElementSaver.convert(ElementSaver.java:250)
> > at
> > uk.me.parabola.mkgmap.reader.osm.o5m.O5mBinMapDataSource.load(O5mBinMapDataSource.java:53)
> > at
> > uk.me.parabola.mkgmap.reader.osm.OsmMapDataSource.load(OsmMapDataSource.java:130)
> > at uk.me.parabola.mkgmap.main.MapMaker.loadFromFile(MapMaker.java:167)
> > at uk.me.parabola.mkgmap.main.MapMaker.makeMap(MapMaker.java:63)
> > at uk.me.parabola.mkgmap.main.Main$1.call(Main.java:253)
> > at uk.me.parabola.mkgmap.main.Main$1.call(Main.java:249)
> > at java.util.concurrent.FutureTask.run(FutureTask.java:262)
> > at
> > java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
> > at
> > java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
> > at java.lang.Thread.run(Thread.java:745)
> >
> >
> > --
> > amarok2 now playing:
> >
> >
> >
> >
> > _______________________________________________
> > mkgmap-dev mailing list
> > mkgmap-dev@lists.mkgmap.org.uk
> > http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
>
> _______________________________________________
> mkgmap-dev mailing list
> mkgmap-dev@lists.mkgmap.org.uk
> http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev