Download
Getting Started
Members
Projects
Community
Marketplace
Events
Planet Eclipse
Newsletter
Videos
Participate
Report a Bug
Forums
Mailing Lists
Wiki
IRC
How to Contribute
Working Groups
Automotive
Internet of Things
LocationTech
Long-Term Support
PolarSys
Science
OpenMDM
More
Community
Marketplace
Events
Planet Eclipse
Newsletter
Videos
Participate
Report a Bug
Forums
Mailing Lists
Wiki
IRC
How to Contribute
Working Groups
Automotive
Internet of Things
LocationTech
Long-Term Support
PolarSys
Science
OpenMDM
Toggle navigation
Bugzilla – Attachment 79308 Details for
Bug 203662
Perf: Unnecessary compilation when package added to second source root
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Requests
|
Help
|
Log In
[x]
|
Terms of Use
|
Copyright Agent
[patch]
Proposed patch
patch.txt (text/plain), 1.71 KB, created by
Kent Johnson
on 2007-09-27 14:46:47 EDT
(
hide
)
Description:
Proposed patch
Filename:
MIME Type:
Creator:
Kent Johnson
Created:
2007-09-27 14:46:47 EDT
Size:
1.71 KB
patch
obsolete
>### Eclipse Workspace Patch 1.0 >#P org.eclipse.jdt.core >Index: model/org/eclipse/jdt/internal/core/builder/IncrementalImageBuilder.java >=================================================================== >RCS file: /cvsroot/eclipse/org.eclipse.jdt.core/model/org/eclipse/jdt/internal/core/builder/IncrementalImageBuilder.java,v >retrieving revision 1.97 >diff -u -r1.97 IncrementalImageBuilder.java >--- model/org/eclipse/jdt/internal/core/builder/IncrementalImageBuilder.java 27 Jun 2007 19:48:44 -0000 1.97 >+++ model/org/eclipse/jdt/internal/core/builder/IncrementalImageBuilder.java 27 Sep 2007 18:45:47 -0000 >@@ -501,10 +501,15 @@ > if (!isExcluded) { > IPath addedPackagePath = resource.getFullPath().removeFirstSegments(segmentCount); > createFolder(addedPackagePath, md.binaryFolder); // ensure package exists in the output folder >- // add dependents even when the package thinks it exists to be on the safe side >- if (JavaBuilder.DEBUG) >- System.out.println("Found added package " + addedPackagePath); //$NON-NLS-1$ >- addDependentsOf(addedPackagePath, true); >+ // see if any known source file is from the same package... classpath already includes new package >+ if (sourceLocations.length > 1 && newState.isKnownPackage(addedPackagePath.toString())) { >+ if (JavaBuilder.DEBUG) >+ System.out.println("Skipped dependents of added package " + addedPackagePath); //$NON-NLS-1$ >+ } else { >+ if (JavaBuilder.DEBUG) >+ System.out.println("Found added package " + addedPackagePath); //$NON-NLS-1$ >+ addDependentsOf(addedPackagePath, true); >+ } > } > // fall thru & collect all the source files > case IResourceDelta.CHANGED :
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
Attachments on
bug 203662
:
78597
| 79308