commit 247474085efd989f67186ca8668a20a72f9ecfb9
author Douglas Gregor <dgregor@apple.com> Mon Aug 16 16:46:30 2010 +0000
committer Douglas Gregor <dgregor@apple.com> Mon Aug 16 16:46:30 2010 +0000
tree 1c25678586409ebca49ee672130e5203f3f7fbf5
parent 57825eea36177da7f9f941bc784d7f3c9824d36e

Dereferencing NULL pointers is such poor form.

llvm-svn: 111150

clang/lib/Frontend/ASTUnit.cpp

diff --git a/clang/lib/Frontend/ASTUnit.cpp b/clang/lib/Frontend/ASTUnit.cpp
index a573fb4..b54162f 100644
--- a/clang/lib/Frontend/ASTUnit.cpp
+++ b/clang/lib/Frontend/ASTUnit.cpp
@@ -191,10 +191,14 @@
                                                         Ctx->getLangOptions());
       CachedResult.Priority = Results[I].Priority;
       CachedResult.Kind = Results[I].CursorKind;
-      CachedResult.TypeClass
-        = getSimplifiedTypeClass(
-              Ctx->getCanonicalType(getDeclUsageType(*Ctx, 
-                                                     Results[I].Declaration)));
+
+      QualType UsageType = getDeclUsageType(*Ctx, Results[I].Declaration);
+      if (UsageType.isNull())
+        CachedResult.TypeClass = STC_Void;
+      else {
+        CachedResult.TypeClass
+          = getSimplifiedTypeClass(Ctx->getCanonicalType(UsageType));
+      }
       CachedCompletionResults.push_back(CachedResult);
       break;
     }