From c0707628e8f4a0086f91102da23751800f223f03 Mon Sep 17 00:00:00 2001
From: Andrey Breslav <andrey.breslav@jetbrains.com>
Date: Tue, 18 Jun 2013 11:45:28 +0400
Subject: [PATCH] DeserializedType uses lazy values

---
 .../frontend/serialization/serialization.iml  |  1 +
 .../serialization/TypeDeserializer.java       | 54 ++++++++++++-------
 2 files changed, 36 insertions(+), 19 deletions(-)

diff --git a/compiler/frontend/serialization/serialization.iml b/compiler/frontend/serialization/serialization.iml
index b5ab6ff7b26..4dd84abaaf1 100644
--- a/compiler/frontend/serialization/serialization.iml
+++ b/compiler/frontend/serialization/serialization.iml
@@ -14,6 +14,7 @@
     <orderEntry type="library" name="annotations" level="project" />
     <orderEntry type="library" name="trove4j" level="project" />
     <orderEntry type="library" name="intellij-core" level="project" />
+    <orderEntry type="module" module-name="util" />
   </component>
 </module>
 
diff --git a/compiler/frontend/serialization/src/org/jetbrains/jet/descriptors/serialization/TypeDeserializer.java b/compiler/frontend/serialization/src/org/jetbrains/jet/descriptors/serialization/TypeDeserializer.java
index bc39d20f040..55b4c136ea0 100644
--- a/compiler/frontend/serialization/src/org/jetbrains/jet/descriptors/serialization/TypeDeserializer.java
+++ b/compiler/frontend/serialization/src/org/jetbrains/jet/descriptors/serialization/TypeDeserializer.java
@@ -23,6 +23,8 @@ import org.jetbrains.jet.lang.descriptors.ClassDescriptor;
 import org.jetbrains.jet.lang.descriptors.ClassifierDescriptor;
 import org.jetbrains.jet.lang.descriptors.TypeParameterDescriptor;
 import org.jetbrains.jet.lang.descriptors.annotations.AnnotationDescriptor;
+import org.jetbrains.jet.lang.resolve.lazy.storage.NotNullLazyValue;
+import org.jetbrains.jet.lang.resolve.lazy.storage.NotNullLazyValueImpl;
 import org.jetbrains.jet.lang.resolve.scopes.JetScope;
 import org.jetbrains.jet.lang.types.*;
 import org.jetbrains.jet.lang.types.checker.JetTypeChecker;
@@ -165,22 +167,34 @@ public class TypeDeserializer {
 
     private class DeserializedType implements JetType {
         private final ProtoBuf.Type typeProto;
-        private TypeConstructor constructor;
+        private final NotNullLazyValue<TypeConstructor> constructor;
         private final List<TypeProjection> arguments;
-        private JetScope memberScope;
+        private final NotNullLazyValue<JetScope> memberScope;
 
-        public DeserializedType(@NotNull ProtoBuf.Type typeProto) {
-            this.typeProto = typeProto;
-            this.arguments = typeArguments(typeProto.getArgumentsList());
+        public DeserializedType(@NotNull ProtoBuf.Type proto) {
+            this.typeProto = proto;
+            this.arguments = typeArguments(proto.getArgumentsList());
+
+            this.constructor = new NotNullLazyValueImpl<TypeConstructor>() {
+                @NotNull
+                @Override
+                protected TypeConstructor doCompute() {
+                    return typeConstructor(typeProto);
+                }
+            };
+            this.memberScope = new NotNullLazyValueImpl<JetScope>() {
+                @NotNull
+                @Override
+                protected JetScope doCompute() {
+                    return computeMemberScope();
+                }
+            };
         }
 
         @NotNull
         @Override
         public TypeConstructor getConstructor() {
-            if (constructor == null) {
-                constructor = typeConstructor(typeProto);
-            }
-            return constructor;
+            return constructor.compute();
         }
 
         @NotNull
@@ -194,19 +208,21 @@ public class TypeDeserializer {
             return typeProto.getNullable();
         }
 
+        @NotNull
+        private JetScope computeMemberScope() {
+            TypeConstructor typeConstructor = getConstructor();
+            if (ErrorUtils.isError(typeConstructor)) {
+                return ErrorUtils.createErrorScope(typeConstructor.toString());
+            }
+            else {
+                return getTypeMemberScope(typeConstructor, getArguments());
+            }
+        }
+
         @NotNull
         @Override
         public JetScope getMemberScope() {
-            if (memberScope == null) {
-                TypeConstructor typeConstructor = getConstructor();
-                if (ErrorUtils.isError(typeConstructor)) {
-                    memberScope = ErrorUtils.createErrorScope(typeConstructor.toString());
-                }
-                else {
-                    memberScope = getTypeMemberScope(typeConstructor, getArguments());
-                }
-            }
-            return memberScope;
+            return memberScope.compute();
         }
 
         @Override