Fixes a potential compilation error.

Pulling the template implementation into the header to guarantee
that it's visible to all possible instantiations.

llvm-svn: 146973

2 files changed, 10 insertions, 13 deletions

diff --git a/llvm/include/llvm/Support/JSONParser.h b/llvm/include/llvm/Support/JSONParser.h
index 0150646bc37..f4cdfa56088 100644
--- a/llvm/include/llvm/Support/JSONParser.h
+++ b/llvm/include/llvm/Support/JSONParser.h
@@ -128,7 +128,16 @@ private:
 
   /// \brief Skips all elements in the given container.
   template <typename ContainerT>
-  bool skipContainer(const ContainerT &Container);
+  bool skipContainer(const ContainerT &Container) {
+    for (typename ContainerT::const_iterator I = Container.current(),
+                                             E = Container.end();
+         I != E; ++I) {
+      assert(*I != 0);
+      if (!skip(**I))
+        return false;
+    }
+    return !failed();
+  }
 
   /// \brief Skips to the next position behind the given JSON atom.
   bool skip(const JSONAtom &Atom);
diff --git a/llvm/lib/Support/JSONParser.cpp b/llvm/lib/Support/JSONParser.cpp
index 02187409cbf..994ca9fc966 100644
--- a/llvm/lib/Support/JSONParser.cpp
+++ b/llvm/lib/Support/JSONParser.cpp
@@ -43,18 +43,6 @@ bool JSONParser::validate() {
   return skip(*parseRoot());
 }
 
-template <typename ContainerT>
-bool JSONParser::skipContainer(const ContainerT &Container) {
-  for (typename ContainerT::const_iterator I = Container.current(),
-                                           E = Container.end();
-       I != E; ++I) {
-    assert(*I != 0);
-    if (!skip(**I))
-      return false;
-  }
-  return !failed();
-}
-
 bool JSONParser::skip(const JSONAtom &Atom) {
   switch(Atom.getKind()) {
     case JSONAtom::JK_Array: return skipContainer(*cast<JSONArray>(&Atom));