Hi, in looking into a possible code change to handle mkgmap:add-to-index=no,
I came across what looks like an error in point.getPOIRecord.

The code is currently:

        public POIRecord getPOIRecord() {
                if (poi == null)
                        return new POIRecord();
                return poi;
        }

Which means if getPOIRecord is called twice, it will return a new POIRecord
each time.
I suspect the code should be:

        public POIRecord getPOIRecord() {
                if (poi == null)
                        poi = new POIRecord();
                return poi;
        }

However, there may be a valid reason for the code being as it is. Any
thoughts?

Regards,
Mike

_______________________________________________
mkgmap-dev mailing list
[email protected]
http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev

Reply via email to