Pixmap symbol scaling bug #1684: review and back porting

[Steve Lime]

Jani: That change was made when I added some code from MapMedia to do symbol rotation. Those folks had made the change in their version and I accepted the change. It is causing a bit of heartburn with folks moving from 4.6 to 4.8 so I would suggest reverting to the old mechanism for 4.8.2. Not sure about CVS head though- I don't mind the change but within major changes in behavior seem to be problematic.

Steve

>>> Jani Averbach <javerbach at EXTENDTHEREACH.COM> 03/02/06 5:38 PM >>>
> -----Original Message-----
> Subject: Re: Pixmap symbol scaling bug #1684: review and back porting
> 
> If there is a change of behavior then the docs will need to 
> be updated to make the new behavior clear and to avoid this becoming 
> another FAQ. In addition to updating the docs, could you please also 
> describe the change of behavior in your HISTORY.TXT entries in 
> both 4.8 and 4.9.

I will do that.

> 
> > If everyone is happy with the fix, I will back port this to 4.8.x 
> > and close the bug #1684.
> > 
> 
> Since there's a change of behavior it's a bit more tricky, 
> but if things were plain broken in 4.8 before then the 
> fix with the new behavior is better than a broken feature, 
> so based on the little information that I 
> have on this I'd say go ahead with backporting.

The behavioral change has been appeared by this this change
(before 4.8):

branch-4-8/mapgd.c: 1.106        (sdlime   14-Oct-05)

   1640       max_size = MS_MAX(symbol->sizex, symbol->sizey);
   1641       d = max_size != 0 ? size/max_size : 1.0; /* was d = size/symbol->sizey; */

I think the new code makes more sense and is just something which 
has changed by 4.8 version. We just have to document this somewhere.
Or revert back to old code (sizey), but that's not my call.

BR, Jani