Index: trunk/src/org/openstreetmap/josm/data/osm/Filters.java =================================================================== --- trunk/src/org/openstreetmap/josm/data/osm/Filters.java (revision 3351) +++ trunk/src/org/openstreetmap/josm/data/osm/Filters.java (revision 3352) @@ -8,5 +8,4 @@ import java.awt.Graphics; import java.awt.Graphics2D; - import java.util.Collection; import java.util.HashSet; @@ -69,58 +68,60 @@ return; - final Collection all = ds.allNonDeletedCompletePrimitives(); - // temporary set to collect the primitives returned by the search engine - final Collection collect = new HashSet(); - - // an auxiliary property to collect the results of the search engine - class CollectProperty implements Property { - boolean collectValue; - boolean hidden; - - /** - * Depending on the parameters, there are 4 different instances - * of this class. - * - * @param collectValue - * If true: collect only those primitives that are added - * by the search engine. - * If false: Collect only those primitives that are removed - * by the search engine. - * @param hidden Whether the property refers to primitives that - * are disabled and hidden or to primitives - * that are disabled only. - */ - public CollectProperty(boolean collectValue, boolean hidden) { - this.collectValue = collectValue; - this.hidden = hidden; - } - - public Boolean get(OsmPrimitive osm) { - if (hidden) - return osm.isDisabledAndHidden(); - else - return osm.isDisabled(); - } - - public void set(OsmPrimitive osm, Boolean value) { - if (collectValue == value.booleanValue()) { - collect.add(osm); + ds.beginUpdate(); // Modifies disabled/hidden state covered by write lock so read lock is not enough + try { + final Collection all = ds.allNonDeletedCompletePrimitives(); + // temporary set to collect the primitives returned by the search engine + final Collection collect = new HashSet(); + + // an auxiliary property to collect the results of the search engine + class CollectProperty implements Property { + boolean collectValue; + boolean hidden; + + /** + * Depending on the parameters, there are 4 different instances + * of this class. + * + * @param collectValue + * If true: collect only those primitives that are added + * by the search engine. + * If false: Collect only those primitives that are removed + * by the search engine. + * @param hidden Whether the property refers to primitives that + * are disabled and hidden or to primitives + * that are disabled only. + */ + public CollectProperty(boolean collectValue, boolean hidden) { + this.collectValue = collectValue; + this.hidden = hidden; } - } - } - - clearFilterFlags(); - - for (Filter flt : filters){ - if (flt.enable) { - collect.clear(); - // Decide, whether primitives are collected that are added to the current - // selection or those that are removed from the current selection - boolean collectValue = flt.mode == SearchAction.SearchMode.replace || flt.mode == SearchAction.SearchMode.add; - Property collectProp = new CollectProperty(collectValue, flt.hiding); - - SearchAction.getSelection(flt, all, collectProp); - - switch (flt.mode) { + + public Boolean get(OsmPrimitive osm) { + if (hidden) + return osm.isDisabledAndHidden(); + else + return osm.isDisabled(); + } + + public void set(OsmPrimitive osm, Boolean value) { + if (collectValue == value.booleanValue()) { + collect.add(osm); + } + } + } + + clearFilterFlags(); + + for (Filter flt : filters){ + if (flt.enable) { + collect.clear(); + // Decide, whether primitives are collected that are added to the current + // selection or those that are removed from the current selection + boolean collectValue = flt.mode == SearchAction.SearchMode.replace || flt.mode == SearchAction.SearchMode.add; + Property collectProp = new CollectProperty(collectValue, flt.hiding); + + SearchAction.getSelection(flt, all, collectProp); + + switch (flt.mode) { case replace: for (OsmPrimitive osm : all) { @@ -137,24 +138,28 @@ if (osm instanceof Way) { nodes: - for (Node n : ((Way)osm).getNodes()) { - // if node is already disabled, there is nothing to do - if (n.isDisabledAndHidden() || (!flt.hiding && n.isDisabled())) - continue; - - // if the node is tagged, don't disable it - if (n.isTagged()) - continue; - - // if the node has undisabled parent ways, don't disable it - for (OsmPrimitive ref : n.getReferrers()) { - if (ref instanceof Way) { - if (!ref.isDisabled()) - continue nodes; - if (flt.hiding && !ref.isDisabledAndHidden()) - continue nodes; + for (Node n : ((Way)osm).getNodes()) { + // if node is already disabled, there is nothing to do + if (n.isDisabledAndHidden() || (!flt.hiding && n.isDisabled())) { + continue; } + + // if the node is tagged, don't disable it + if (n.isTagged()) { + continue; + } + + // if the node has undisabled parent ways, don't disable it + for (OsmPrimitive ref : n.getReferrers()) { + if (ref instanceof Way) { + if (!ref.isDisabled()) { + continue nodes; + } + if (flt.hiding && !ref.isDisabledAndHidden()) { + continue nodes; + } + } + } + n.setDisabledState(flt.hiding); } - n.setDisabledState(flt.hiding); - } } } @@ -169,22 +174,25 @@ // update flags for nodes nodes: - for (OsmPrimitive osm : collect) { - if (osm instanceof Node) { - // if node is already disabled, there is nothing to do - if (osm.isDisabledAndHidden() || (!flt.hiding && osm.isDisabled())) - continue; - - // if the node has undisabled parent ways, don't disable it - for (OsmPrimitive ref : osm.getReferrers()) { - if (ref instanceof Way) { - if (!ref.isDisabled()) - continue nodes; - if (flt.hiding && !ref.isDisabledAndHidden()) - continue nodes; + for (OsmPrimitive osm : collect) { + if (osm instanceof Node) { + // if node is already disabled, there is nothing to do + if (osm.isDisabledAndHidden() || (!flt.hiding && osm.isDisabled())) { + continue; } + + // if the node has undisabled parent ways, don't disable it + for (OsmPrimitive ref : osm.getReferrers()) { + if (ref instanceof Way) { + if (!ref.isDisabled()) { + continue nodes; + } + if (flt.hiding && !ref.isDisabledAndHidden()) { + continue nodes; + } + } + } + osm.setDisabledState(flt.hiding); } - osm.setDisabledState(flt.hiding); } - } } break; @@ -223,30 +231,33 @@ default: throw new IllegalStateException(); + } } } - } - - disabledCount = 0; - disabledAndHiddenCount = 0; - // collect disabled and selected the primitives - final Collection deselect = new HashSet(); - for (OsmPrimitive osm : all) { - if (osm.isDisabled()) { - disabledCount++; - if (osm.isSelected()) { - deselect.add(osm); + + disabledCount = 0; + disabledAndHiddenCount = 0; + // collect disabled and selected the primitives + final Collection deselect = new HashSet(); + for (OsmPrimitive osm : all) { + if (osm.isDisabled()) { + disabledCount++; + if (osm.isSelected()) { + deselect.add(osm); + } + if (osm.isDisabledAndHidden()) { + disabledAndHiddenCount++; + } } - if (osm.isDisabledAndHidden()) { - disabledAndHiddenCount++; - } - } - } - disabledCount -= disabledAndHiddenCount; - if (!deselect.isEmpty()) { - ds.clearSelection(deselect); - } - - Main.map.mapView.repaint(); - Main.map.filterDialog.updateDialogHeader(); + } + disabledCount -= disabledAndHiddenCount; + if (!deselect.isEmpty()) { + ds.clearSelection(deselect); + } + + Main.map.mapView.repaint(); + Main.map.filterDialog.updateDialogHeader(); + } finally { + ds.endUpdate(); + } }