Hi Alberto, don't think that this should be a problem.
Question is why was it static before? Maybe an error.

I am using raster stuff a bit with my HoRAE extension... so it would be 
necessary for me to test if still everything works "after" you made your 
changes. (I write "after", because I am actually not having time the 
next 2 weeks)

cheers,
stefan

Am 06.10.14 07:34, schrieb Alberto De Luca - GeA:
> 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
> 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

Reply via email to