Changes between Version 2 and Version 3 of Ticket #15625, comment 6


Ignore:
Timestamp:
2017-12-03T11:11:19+01:00 (6 years ago)
Author:
cmuelle8

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #15625, comment 6

    v2 v3  
    1717This may not be a regression. Have you checked that the test image in comment:3 really works with older versions (e.g. r13037 or earlier)?
    1818
    19 Since {{{test-landscape.jpg}}} and {{{test-portrait.jpg}}} work, I suspect to be the root cause to be within the range of [source:trunk/src/org/openstreetmap/josm/gui/layer/geoimage/ImageDisplay.java?rev=13186#L251 line 251] and [source:trunk/src/org/openstreetmap/josm/gui/layer/geoimage/ImageDisplay.java?rev=13186#L269 line 269] of [source:trunk/src/org/openstreetmap/josm/gui/layer/geoimage/ImageDisplay.java?rev=13186].
     19Since {{{test-landscape.jpg}}} and {{{test-portrait.jpg}}} work, I suspect the root cause to be within the range of [source:trunk/src/org/openstreetmap/josm/gui/layer/geoimage/ImageDisplay.java?rev=13186#L251 line 251] and [source:trunk/src/org/openstreetmap/josm/gui/layer/geoimage/ImageDisplay.java?rev=13186#L269 line 269] of [source:trunk/src/org/openstreetmap/josm/gui/layer/geoimage/ImageDisplay.java?rev=13186].
    2020
    2121It's probably better to create a new VisRect there (so that init is set correctly), instead of just swapping width and height of the one already created at this point.  I'll update the attachments of ticket:15574 to contain this fix within some minutes, once I've tested the fix.