Re: [JPP-Devel] LayerManager and org.openjump.core.ccordsys.srid.SRIDStyle

[Larry B. <bec...@gm...>]

Hi Volker,

  I took a look at the SRIDStyle code, but I'm just not familiar enough with
it to confirm or deny your conclusions.  LayerListeners are confusing and I
avoid them whenever possible.  SkyJUMP doesn't use SRIDStyle and I'm not
sure why it was implemented this way in OJ.  Perhaps the author or someone
more familiar with it will respond.

regards,
Larry

On Fri, Mar 26, 2010 at 9:09 AM, <vow...@ao...> wrote:

>  Hi *,
>
> Each instance of SRIDStyle adds one LayerListener to the LayerManager ( and
> does not remove it, when its layer is removed, I guess).
>
> My understanding is that each Layer has it own SRIDStyle ( and exactly one,
> if I read EnsureAllLayersHaveSRIDStylePlugIn.ensureHasSRIDStyle correctly )
> and that each LayerListener is called when
> LayerManager.fireFeaturesChanged is called.
>
> So if there are 10 Layers and one Feature of Layer X is changed then 10
> LayerListener are executed and each LayerListener change the Srid of the
> Geometry of the changed Feature. The Srid of the feature after that, is the
> Srid of the last LayerListener of the ArrayList LayerManager.layerListeners.
>
>
> The "easiest" change would be that the sridstyle knows its layer and only
> changes the srid of FeatureEvent where the layer matches its own.
>
> Any opinons about that "issue"?
>
> regards,
>
> Volker
>
>
>
>
>
> ------------------------------------------------------------------------------
> Download Intel&#174; Parallel Studio Eval
> Try the new software tools for yourself. Speed compiling, find bugs
> proactively, and fine-tune applications for parallel performance.
> See why Intel Parallel Studio got high marks during beta.
> http://p.sf.net/sfu/intel-sw-dev
> _______________________________________________
> Jump-pilot-devel mailing list
> Jum...@li...
> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>
>