Improvement to getCustomOptions to prevent premature loading of element info

Signed-off-by: Rob Stryker <[email protected]>

Author: robstryker

org.eclipse.jdt.core/model/org/eclipse/jdt/internal/core/CompilationUnit.java

@@ -1579,8 +1579,16 @@ public void setOptions(Map<String, String> newOptions) {
 @Override
 public Map<String, String> getCustomOptions() {
 	if (this.owner != null) {
-		try {
-			Map<String, String> customOptions = this.getCompilationUnitElementInfo().getCustomOptions();
+//		try {
+			Map<String, String> customOptions = null;
+			JavaModelManager manager = JavaModelManager.getJavaModelManager();
+			IElementInfo info = manager.getInfo(this);
+			// If the info has not been loaded, then nobody has called setOptions() yet.
+			// So no need to load the IElementInfo, which may also parse the cu.
+			if (info != null)
+				customOptions = ((CompilationUnitElementInfo)info).getCustomOptions();
+
+//			Map<String, String> customOptions = this.getCompilationUnitElementInfo().getCustomOptions();
 			IJavaProject parentProject = getJavaProject();
 			Map<String, String> parentOptions = parentProject == null ? JavaCore.getOptions() : parentProject.getOptions(true);
 			if (JavaCore.ENABLED.equals(parentOptions.get(JavaCore.COMPILER_PB_ENABLE_PREVIEW_FEATURES)) &&
@@ -1593,9 +1601,9 @@ public Map<String, String> getCustomOptions() {
 				}
 			}
 			return customOptions == null ? Collections.emptyMap() : customOptions;
-		} catch (JavaModelException e) {
-			// do nothing
-		}
+//		} catch (JavaModelException e) {
+//			// do nothing
+//		}
 	}
 
 	return Collections.emptyMap();