[mkgmap-dev] point.getPOIRecord

Thu Feb 5 19:23:31 GMT 2015

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