Hi,

What difference this change could mean and how to test?

-Jukka Rahkonen-

Michael Michaud wrote:

Hi Alberto, Stefan,

Just commit Alberto's patch (r4064)
please test,

Michaƫl
Dear OJ developers,

after a break (of a few years...), I'm back on OJ development, still focussed 
on raster management and close to be ready to make available to the community 
an improved version of the "raster color editor", allowing the use of ramps, 
intervals, single colour values...

Nevertheless, there is a design issue in the class

org.openjump.core.rasterimage.RasterImageLayer

that is giving me some problems. The point is that the Raster variable 
"rasterData" is static. As a consequence, it gets shared across all the 
instances of RasterImageLayer, making thinks a bit awkward. Do you see any 
problem in turning rasterData into non-static? I've tested this change, and 
everything appears to be working alright. You can find attached the modified 
RasterImageLayer.java

Cheers
Alberto




------------------------------------------------------------------------------

Slashdot TV.  Videos for Nerds.  Stuff that Matters.

http://pubads.g.doubleclick.net/gampad/clk?id=160591471&iu=/4140/ostg.clktrk




_______________________________________________

Jump-pilot-devel mailing list

Jump-pilot-devel@lists.sourceforge.net<mailto:Jump-pilot-devel@lists.sourceforge.net>

https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

------------------------------------------------------------------------------
Meet PCI DSS 3.0 Compliance Requirements with EventLog Analyzer
Achieve PCI DSS 3.0 Compliant Status with Out-of-the-box PCI DSS Reports
Are you Audit-Ready for PCI DSS 3.0 Compliance? Download White paper
Comply to PCI DSS 3.0 Requirement 10 and 11.5 with EventLog Analyzer
http://pubads.g.doubleclick.net/gampad/clk?id=154622311&iu=/4140/ostg.clktrk
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to