Hi Alberto,
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...
Good news ;-)
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
Generally speaking, it seems a bad idea to put heavy structures in a
static field.
Wonder how it could work. The best is to commit your code and to validate it
with some tests including heapsize measurement as suggested by Ede.
Michaël
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
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
------------------------------------------------------------------------------
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
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
