hey Mike,

On 04.06.2018 00:37, jump-pilot-svn--- via Jump-pilot-devel wrote:
> +            if (e.getType() == LayerEventType.METADATA_CHANGED) {
> +              //If layer becomes editable, apply row striping
> +              // and remove gridlines,
> +              //as recommended in Java Look and Feel Design
> +              // Guidelines: Advanced Topics [Jon Aquino]
> +              updateGrid(model.getLayer());
> +              repaint();
> +            }
> +            if (e.getType() == LayerEventType.REMOVED) {
> +              e.getLayerable().getLayerManager().removeLayerListener(this);
> +            }

any objection on me changeing the second if to an 'else if' so not both 
conditions are tested every time?
or maybe i'll add a return in each, which would have the same effect.

 ..ede

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to