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