From: Daniel Jasper Date: Tue, 8 Nov 2016 19:47:19 +0000 (+0000) Subject: Remove FormatContext from libClang as it is now unused. X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=b74377b0a34a33cb3c7b565f0e543cd3e56b3f6b;p=clang Remove FormatContext from libClang as it is now unused. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@286279 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/include/clang/Index/CommentToXML.h b/include/clang/Index/CommentToXML.h index bb7b71ad41..04f9501288 100644 --- a/include/clang/Index/CommentToXML.h +++ b/include/clang/Index/CommentToXML.h @@ -22,12 +22,7 @@ class HTMLTagComment; } namespace index { -class SimpleFormatContext; - class CommentToXMLConverter { - std::unique_ptr FormatContext; - unsigned FormatInMemoryUniqueId; - public: CommentToXMLConverter(); ~CommentToXMLConverter(); diff --git a/lib/Index/CommentToXML.cpp b/lib/Index/CommentToXML.cpp index 4210b897c3..ee066cc6d9 100644 --- a/lib/Index/CommentToXML.cpp +++ b/lib/Index/CommentToXML.cpp @@ -8,7 +8,6 @@ //===----------------------------------------------------------------------===// #include "clang/Index/CommentToXML.h" -#include "SimpleFormatContext.h" #include "clang/AST/ASTContext.h" #include "clang/AST/Attr.h" #include "clang/AST/Comment.h" @@ -531,12 +530,8 @@ public: CommentASTToXMLConverter(const FullComment *FC, SmallVectorImpl &Str, const CommandTraits &Traits, - const SourceManager &SM, - SimpleFormatContext &SFC, - unsigned FUID) : - FC(FC), Result(Str), Traits(Traits), SM(SM), - FormatRewriterContext(SFC), - FormatInMemoryUniqueId(FUID) { } + const SourceManager &SM) : + FC(FC), Result(Str), Traits(Traits), SM(SM) { } // Inline content. void visitTextComment(const TextComment *C); @@ -574,8 +569,6 @@ private: const CommandTraits &Traits; const SourceManager &SM; - SimpleFormatContext &FormatRewriterContext; - unsigned FormatInMemoryUniqueId; }; void getSourceTextOfDeclaration(const DeclInfo *ThisDecl, @@ -596,18 +589,13 @@ void CommentASTToXMLConverter::formatTextOfDeclaration( StringRef StringDecl(Declaration.c_str(), Declaration.size()); // Formatter specific code. - // Form a unique in memory buffer name. - SmallString<128> Filename; - Filename += "xmldecl"; - Filename += llvm::utostr(FormatInMemoryUniqueId); - Filename += ".xd"; unsigned Offset = 0; unsigned Length = Declaration.size(); bool IncompleteFormat = false; tooling::Replacements Replaces = reformat(format::getLLVMStyle(), StringDecl, - tooling::Range(Offset, Length), Filename, &IncompleteFormat); + tooling::Range(Offset, Length), "xmldecl.xd", &IncompleteFormat); auto FormattedStringDecl = applyAllReplacements(StringDecl, Replaces); if (static_cast(FormattedStringDecl)) { Declaration = *FormattedStringDecl; @@ -1127,7 +1115,7 @@ void CommentASTToXMLConverter::appendToResultWithCDATAEscaping(StringRef S) { Result << "]]>"; } -CommentToXMLConverter::CommentToXMLConverter() : FormatInMemoryUniqueId(0) {} +CommentToXMLConverter::CommentToXMLConverter() {} CommentToXMLConverter::~CommentToXMLConverter() {} void CommentToXMLConverter::convertCommentToHTML(const FullComment *FC, @@ -1149,14 +1137,7 @@ void CommentToXMLConverter::convertHTMLTagNodeToText( void CommentToXMLConverter::convertCommentToXML(const FullComment *FC, SmallVectorImpl &XML, const ASTContext &Context) { - if (!FormatContext || (FormatInMemoryUniqueId % 1000) == 0) { - // Create a new format context, or re-create it after some number of - // iterations, so the buffers don't grow too large. - FormatContext.reset(new SimpleFormatContext(Context.getLangOpts())); - } - CommentASTToXMLConverter Converter(FC, XML, Context.getCommentCommandTraits(), - Context.getSourceManager(), *FormatContext, - FormatInMemoryUniqueId++); + Context.getSourceManager()); Converter.visit(FC); }