Attachment 177612 Details for Bug 307587 – Proposal v03

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

[patch] Proposal v03

307587_20100827-1425.txt (text/plain), 8.96 KB, created by Szymon Brandys

on 2010-08-27 08:26:01 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Szymon Brandys

Created: 2010-08-27 08:26:01 EDT

Size: 8.96 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.core.resources
>Index: src/org/eclipse/core/internal/resources/NatureManager.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.core.resources/src/org/eclipse/core/internal/resources/NatureManager.java,v
>retrieving revision 1.30
>diff -u -r1.30 NatureManager.java
>--- src/org/eclipse/core/internal/resources/NatureManager.java	3 Dec 2007 04:54:22 -0000	1.30
>+++ src/org/eclipse/core/internal/resources/NatureManager.java	27 Aug 2010 12:25:23 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2007 IBM Corporation and others.
>+ * Copyright (c) 2000, 2010 IBM Corporation and others.
>  * All rights reserved. This program and the accompanying materials
>  * are made available under the terms of the Eclipse Public License v1.0
>  * which accompanies this distribution, and is available at
>@@ -25,14 +25,14 @@
>  */
> public class NatureManager implements ILifecycleListener, IManager {
> 	//maps String (nature ID) -> descriptor objects
>-	protected Map descriptors;
>+	private Map descriptors;
> 
> 	//maps IProject -> String[] of enabled natures for that project
>-	protected Map natureEnablements;
>+	private Map natureEnablements;
> 
> 	//maps String (builder ID) -> String (nature ID)
>-	protected Map buildersToNatures = null;
>-	//colour constants used in cycle detection algorithm
>+	private Map buildersToNatures = null;
>+	//color constants used in cycle detection algorithm
> 	private static final byte WHITE = 0;
> 	private static final byte GREY = 1;
> 	private static final byte BLACK = 2;
>@@ -43,10 +43,10 @@
> 
> 	/**
> 	 * Computes the list of natures that are enabled for the given project.
>-	 * Enablement computation is subtlely different from nature set
>+	 * Enablement computation is subtlety different from nature set
> 	 * validation, because it must find and remove all inconsistencies.
> 	 */
>-	protected String[] computeNatureEnablements(Project project) {
>+	private String[] computeNatureEnablements(Project project) {
> 		final ProjectDescription description = project.internalGetDescription();
> 		if (description == null)
> 			return new String[0];//project deleted concurrently
>@@ -136,7 +136,7 @@
> 	/**
> 	 * Configures the nature with the given ID for the given project.
> 	 */
>-	protected void configureNature(final Project project, final String natureID, final MultiStatus errors) {
>+	private void configureNature(final Project project, final String natureID, final MultiStatus errors) {
> 		ISafeRunnable code = new ISafeRunnable() {
> 			public void run() throws Exception {
> 				IProjectNature nature = createNature(project, natureID);
>@@ -164,7 +164,7 @@
> 	 * new description are removed.  Updates the old description so that it reflects
> 	 * the new set of the natures.  Errors are added to the given multi-status.
> 	 */
>-	public void configureNatures(Project project, ProjectDescription oldDescription, ProjectDescription newDescription, MultiStatus status) {
>+	public synchronized void configureNatures(Project project, ProjectDescription oldDescription, ProjectDescription newDescription, MultiStatus status) {
> 		// Be careful not to rely on much state because (de)configuring a nature
> 		// may well result in recursive calls to this method.
> 		HashSet oldNatures = new HashSet(Arrays.asList(oldDescription.getNatureIds(false)));
>@@ -240,7 +240,7 @@
> 	/**
> 	 * Deconfigures the nature with the given ID for the given project.
> 	 */
>-	protected void deconfigureNature(final Project project, final String natureID, final MultiStatus status) {
>+	private void deconfigureNature(final Project project, final String natureID, final MultiStatus status) {
> 		final ProjectInfo info = (ProjectInfo) project.getResourceInfo(false, true);
> 		IProjectNature existingNature = info.getNature(natureID);
> 		if (existingNature == null) {
>@@ -275,7 +275,7 @@
> 	/**
> 	 * Marks all nature descriptors that are involved in cycles
> 	 */
>-	protected void detectCycles() {
>+	private void detectCycles() {
> 		Collection values = descriptors.values();
> 		ProjectNatureDescriptor[] natures = (ProjectNatureDescriptor[]) values.toArray(new ProjectNatureDescriptor[values.size()]);
> 		for (int i = 0; i < natures.length; i++)
>@@ -286,7 +286,7 @@
> 	/**
> 	 * Returns a status indicating failure to configure natures.
> 	 */
>-	protected IStatus failure(String reason) {
>+	private IStatus failure(String reason) {
> 		return new ResourceStatus(IResourceStatus.INVALID_NATURE_SET, reason);
> 	}
> 
>@@ -294,7 +294,7 @@
> 	 * Returns the ID of the project nature that claims ownership of the
> 	 * builder with the given ID.  Returns null if no nature owns that builder.
> 	 */
>-	public String findNatureForBuilder(String builderID) {
>+	public synchronized String findNatureForBuilder(String builderID) {
> 		if (buildersToNatures == null) {
> 			buildersToNatures = new HashMap(10);
> 			IProjectNatureDescriptor[] descs = getNatureDescriptors();
>@@ -310,7 +310,7 @@
> 		return (String) buildersToNatures.get(builderID);
> 	}
> 
>-	protected void flushEnablements(IProject project) {
>+	private synchronized void flushEnablements(IProject project) {
> 		if (natureEnablements != null) {
> 			natureEnablements.remove(project);
> 			if (natureEnablements.size() == 0) {
>@@ -323,7 +323,7 @@
> 	 * Returns the cached array of enabled natures for this project,
> 	 * or null if there is nothing in the cache.
> 	 */
>-	protected String[] getEnabledNatures(Project project) {
>+	protected synchronized String[] getEnabledNatures(Project project) {
> 		String[] enabled;
> 		if (natureEnablements != null) {
> 			enabled = (String[]) natureEnablements.get(project);
>@@ -340,7 +340,7 @@
> 	 * dependencies starting at root i.  Returns false otherwise.
> 	 * Marks all descriptors that are involved in the cycle as invalid.
> 	 */
>-	protected boolean hasCycles(ProjectNatureDescriptor desc) {
>+	private boolean hasCycles(ProjectNatureDescriptor desc) {
> 		if (desc.colour == BLACK) {
> 			//this subgraph has already been traversed, so we know the answer
> 			return desc.hasCycle;
>@@ -373,7 +373,7 @@
> 	/**
> 	 * Returns true if the given project has linked resources, and false otherwise.
> 	 */
>-	protected boolean hasLinks(IProject project) {
>+	private boolean hasLinks(IProject project) {
> 		try {
> 			IResource[] children = project.members();
> 			for (int i = 0; i < children.length; i++)
>@@ -391,7 +391,7 @@
> 	 * memberships.  Returns the name of one such overlap, or null if
> 	 * there is no set overlap.
> 	 */
>-	protected String hasSetOverlap(IProjectNatureDescriptor one, IProjectNatureDescriptor two) {
>+	private String hasSetOverlap(IProjectNatureDescriptor one, IProjectNatureDescriptor two) {
> 		if (one == null || two == null) {
> 			return null;
> 		}
>@@ -411,7 +411,7 @@
> 	/**
> 	 * Perform depth-first insertion of the given nature ID into the result list.
> 	 */
>-	protected void insert(ArrayList list, Set seen, String id) {
>+	private void insert(ArrayList list, Set seen, String id) {
> 		if (seen.contains(id))
> 			return;
> 		seen.add(id);
>@@ -443,7 +443,7 @@
> 	 * Only initialize the descriptor cache when we know it is actually needed.
> 	 * Running programs may never need to refer to this cache.
> 	 */
>-	protected void lazyInitialize() {
>+	private synchronized void lazyInitialize() {
> 		if (descriptors != null)
> 			return;
> 		IExtensionPoint point = Platform.getExtensionRegistry().getExtensionPoint(ResourcesPlugin.PI_RESOURCES, ResourcesPlugin.PT_NATURES);
>@@ -467,7 +467,7 @@
> 	/**
> 	 * Sets the cached array of enabled natures for this project.
> 	 */
>-	protected void setEnabledNatures(IProject project, String[] enablements) {
>+	private void setEnabledNatures(IProject project, String[] enablements) {
> 		if (natureEnablements == null)
> 			natureEnablements = Collections.synchronizedMap(new HashMap(20));
> 		natureEnablements.put(project, enablements);
>@@ -513,7 +513,7 @@
> 	 * @return An OK status if all additions are valid, and an error status 
> 	 * 	if any of the additions introduce new inconsistencies.
> 	 */
>-	protected IStatus validateAdditions(HashSet newNatures, HashSet additions, IProject project) {
>+	private IStatus validateAdditions(HashSet newNatures, HashSet additions, IProject project) {
> 		Boolean hasLinks = null;//three states: true, false, null (not yet computed)
> 		//perform checks in order from least expensive to most expensive
> 		for (Iterator added = additions.iterator(); added.hasNext();) {
>@@ -585,7 +585,7 @@
> 	 * @return An OK status if all removals are valid, and a not OK status 
> 	 * 	if any of the deletions introduce new inconsistencies.
> 	 */
>-	protected IStatus validateRemovals(HashSet newNatures, HashSet deletions) {
>+	private IStatus validateRemovals(HashSet newNatures, HashSet deletions) {
> 		//iterate over new nature set, and ensure that none of their prerequisites are being deleted
> 		for (Iterator it = newNatures.iterator(); it.hasNext();) {
> 			String currentID = (String) it.next();

Actions: View | Diff

Attachments on bug 307587: 177513 | 177601 | 177612 | 177643 | 185251 | 185303 | 188503 | 188516 | 189502 | 189520