https://issues.apache.org/bugzilla/show_bug.cgi?id=44751





--- Comment #3 from HervĂ© Girod <[email protected]>  2009-07-05 
02:16:48 PST ---
(In reply to comment #2)
> I will send a patch as soon as I can, but these lines in WMFRecordStore:
> 
> if (_bext && functionId == WMFConstants.META_SETWINDOWEXT) {
>    vpW = width;
>    vpH = height;
>    if (! isotropic) scaleXY = (float)vpW / (float)vpH;
>    vpW = (int)(vpW * scaleXY);                      
>    _bext = false;
> }
> 
> should be changed to:
> 
> if (functionId == WMFConstants.META_SETWINDOWEXT) {
>    if ((width > 0) && (height > 0)) {
>        vpW = width;
>        vpH = height;
>        if (! isotropic) scaleXY = (float)vpW / (float)vpH;
>        vpW = (int)(vpW * scaleXY);  
>    }                    
> }
> 
> Of course, _bext is not useful anymore. The check for width and height being
> greater than 0 is "just in case". There are so many faulty WMF files in the
> wild ;)

Sorry answered for the wrong bug...

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to