From d1707c02c1d49567461a9042e90311041b424ad9 Mon Sep 17 00:00:00 2001
From: Doug Gregor <dgregor@apple.com>
Date: Mon, 11 Jan 2016 21:07:21 -0800
Subject: [PATCH] Revert "Remove unnecessary 'visitDecl' default cases."

---
 tools/SourceKit/lib/SwiftLang/SwiftLangSupport.cpp | 3 +++
 tools/swift-ide-test/ModuleAPIDiff.cpp             | 4 ++++
 2 files changed, 7 insertions(+)

diff --git a/tools/SourceKit/lib/SwiftLang/SwiftLangSupport.cpp b/tools/SourceKit/lib/SwiftLang/SwiftLangSupport.cpp
index ce21f2d863621..0062067d0932b 100644
--- a/tools/SourceKit/lib/SwiftLang/SwiftLangSupport.cpp
+++ b/tools/SourceKit/lib/SwiftLang/SwiftLangSupport.cpp
@@ -156,6 +156,9 @@ class UIdentVisitor : public ASTVisitor<UIdentVisitor,
 public:
   explicit UIdentVisitor(bool IsRef) : IsRef(IsRef) { }
 
+  /// TODO: reconsider whether having a default case is a good idea.
+  UIdent visitDecl(const Decl *D) { return UIdent(); }
+
   UIdent visitFuncDecl(const FuncDecl *D);
   UIdent visitVarDecl(const VarDecl *D);
   UIdent visitExtensionDecl(const ExtensionDecl *D);
diff --git a/tools/swift-ide-test/ModuleAPIDiff.cpp b/tools/swift-ide-test/ModuleAPIDiff.cpp
index 76c45ac49ccc0..3a99f4679bb5c 100644
--- a/tools/swift-ide-test/ModuleAPIDiff.cpp
+++ b/tools/swift-ide-test/ModuleAPIDiff.cpp
@@ -782,6 +782,10 @@ class SMAModelGenerator : public DeclVisitor<SMAModelGenerator> {
     return Result;
   }
 
+  void visitDecl(Decl *D) {
+    // FIXME: maybe don't have a default case
+  }
+
   void visitStructDecl(StructDecl *SD) {
     auto ResultSD = std::make_shared<sma::StructDecl>();
     ResultSD->Name = convertToIdentifier(SD->getName());