On 06.10.2014 13:34, Alberto De Luca - GeA wrote:
> 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...

yippee.. \o/

> 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

hmm.. usually one Layer object represents this layer's data. how did it work 
before, if more than one layer always overwrote the value of the static 
'rasterData' of the older layer?

making it unstatic could raise memory constraints, can you make a test with big 
images, before and after and see if there is a significant difference?

finally.. we have a second image framework, search class ReferencedImagesLayer 
. could you please have a look if your plugin might be able to handle those as 
well?

regards.. ede

------------------------------------------------------------------------------
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

Reply via email to