[mkgmap-dev] point.getPOIRecord

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