Ede,
yes I am aware of the majority of the additions relate to Layer.java, but I
wouldn't venture into commenting on those since my knowledge with regards
to the Layer class is limited. What I could say though is that for what
I've seen the design of Layer is very neat and though-through in
thx! and exactly my point :) ..ede
On 27.01.2016 12:17, Alberto De Luca wrote:
> Ede,
>
> yes I am aware of the majority of the additions relate to Layer.java, but I
> wouldn't venture into commenting on those since my knowledge with regards
> to the Layer class is limited. What I could say
On 26.01.2016 17:31, Giuseppe Aruta wrote:
> Just few extra details:
>
> 1) Temporary vector (Layer.class) and raster (RasterImageLayer.class)
> layers (isTemporaryLayer())
> - Sextante is a different software and it handles raster and vector in a
> separate way from OpenJUMP
> - while TEMP files
Hey guys,
at the moment RasterImageLayer is quite a messy class (a mess to whom I've
been contributing, by the way), because there's a bit of everything in in,
from methods that manipulate the raster values to methods that take care of
the rendering on screen...
So while we wait for the lucky
Alberto,
you are aware most additions Peppe did were to Layer.java where they definitely
don't belong?
if it's needed in RasterImageLayer, i don't oppose at all.
..ede
On 27.01.2016 10:53, Alberto De Luca wrote:
> Hey guys,
>
> at the moment RasterImageLayer is quite a messy class (a mess to
Hi,
I think that Peppe's request show some flaws in layerable hierarchy.
Some may be addressed by adding helper methods to existing class.
Others maybe addressed by introducing new interfaces.
Some are not so difficult to find in current implementation.
My main complain about the design is that
OK,
I had the response of everybody.
- I am going to remove all the Layer.class parts and restore as it is before
- I am going to leave only the RasterImageLayer.class method
isTemporaryLayer()
- I will create an external class, called maybe LayerbleUtils - no
implementation to Layerable or other
Revision: 4803
http://sourceforge.net/p/jump-pilot/code/4803
Author: ma15569
Date: 2016-01-27 15:06:17 + (Wed, 27 Jan 2016)
Log Message:
---
Restored Layer class to previous 4801 version
Correct bug on methods of RasterImageLayer class
Modified Paths:
--