#2756 closed defect (fixed)
AGG: label bbox computation bug (msGetLabelSizeAGG)
Reported by: | aalbarello | Owned by: | tbonfort |
---|---|---|---|
Priority: | normal | Milestone: | 5.2.1 release |
Component: | AGG | Version: | 5.2 |
Severity: | normal | Keywords: | |
Cc: |
Description
Label bboxes on layers having a scalefactor different from 1 (i.e. SIZEUNITS set to real world units instead of pixels) aren't computed correctly inside msGetLabelSizeAGG in mapagg.cpp. In fact, line 2047 of mapagg.cpp (trunk) should quite likely read
if (ren->getLabelSize(string, font, size, rect) != MS_SUCCESS)
instead of
if (ren->getLabelSize(string, font, label->size, rect) != MS_SUCCESS)
as it is right now, thus taking into account the actual label size already calculated on lines 2039-2041.
Andreas
Change History (3)
comment:1 by , 16 years ago
Status: | new → assigned |
---|---|
Summary: | AGG: serious label bbox computation bug (msGetLabelSizeAGG) → AGG: label bbox computation bug (msGetLabelSizeAGG) |
Version: | svn-trunk (development) → 5.2 |
comment:2 by , 16 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:3 by , 16 years ago
Sorry if "serious" in the summary sounded too alarming, but the bug resulted in collision avoidance kicking in all the time and my map having no labels at all, so I thought I'd give this a prominent title ;)
Anyway, thanks for fixing it, Thomas.
fixed in r7891 (5.2) and r7892 (trunk)
thanks Andreas !