Changes in FixItRewriter.cpp

adcd0268 - Make llvm::StringRef to std::string conversions explicit.

Tue, 28 Jan 2020 19:23:46 +0000

Make llvm::StringRef to std::string conversions explicit.This is how it should've been and brings it more in line withstd::string_view. There should be no functional change here.This is mostly mechanical from a custom clang-tidy check, with a lot ofmanual fixups. It uncovers a lot of minor inefficiencies.This doesn't actually modify StringRef yet, I'll do that in a follow-up. List of files: /llvm-project-15.0.7/clang/lib/Frontend/Rewrite/FixItRewriter.cpp

d9b948b6 - Rename F_{None,Text,Append} to OF_{None,Text,Append}. NFC

Mon, 05 Aug 2019 05:43:48 +0000

Rename F_{None,Text,Append} to OF_{None,Text,Append}. NFCF_{None,Text,Append} are kept for compatibility since r334221.llvm-svn: 367800 List of files: /llvm-project-15.0.7/clang/lib/Frontend/Rewrite/FixItRewriter.cpp

2946cd70 - Update the file headers across all of the LLVM projects in the monorepo

Sat, 19 Jan 2019 08:50:56 +0000

Update the file headers across all of the LLVM projects in the monorepoto reflect the new license.We understand that people may be surprised that we're moving the headerentirely to discuss the new license. We checked this carefully with theFoundation's lawyer and we believe this is the correct approach.Essentially, all code in the project is now made available by the LLVMproject under our new license, so you will see that the license headersinclude that license only. Some of our contributors have contributedcode under our old license, and accordingly, we have retained a copy ofour old license notice in the top-level files in each project andrepository.llvm-svn: 351636 List of files: /llvm-project-15.0.7/clang/lib/Frontend/Rewrite/FixItRewriter.cpp

6907ce2f - Remove trailing space

Mon, 30 Jul 2018 19:24:48 +0000

Remove trailing spacesed -Ei 's/[[:space:]]+$//' include/**/*.{def,h,td} lib/**/*.{cpp,h}llvm-svn: 338291 List of files: /llvm-project-15.0.7/clang/lib/Frontend/Rewrite/FixItRewriter.cpp

9fc8faf9 - Remove \brief commands from doxygen comments.

Wed, 09 May 2018 01:00:01 +0000

Remove \brief commands from doxygen comments.This is similar to the LLVM change https://reviews.llvm.org/D46290.We've been running doxygen with the autobrief option for a couple ofyears now. This makes the \brief markers into our commentsredundant. Since they are a visual distraction and we don't want toencourage more \brief markers in new code either, this patch removesthem all.Patch produced byfor i in $(git grep -l '\@brief'); do perl -pi -e 's/\@brief //g' $i & donefor i in $(git grep -l '\\brief'); do perl -pi -e 's/\\brief //g' $i & doneDifferential Revision: https://reviews.llvm.org/D46320llvm-svn: 331834 List of files: /llvm-project-15.0.7/clang/lib/Frontend/Rewrite/FixItRewriter.cpp

db914a46 - [Edit, Rewrite] Fix some Clang-tidy modernize and Include What You Use warnings; other minor fixes (NFC).

Tue, 27 Mar 2018 00:01:49 +0000

[Edit, Rewrite] Fix some Clang-tidy modernize and Include What You Use warnings; other minor fixes (NFC).llvm-svn: 328597 List of files: /llvm-project-15.0.7/clang/lib/Frontend/Rewrite/FixItRewriter.cpp

637d1e66 - Roll-back r250822.

Tue, 20 Oct 2015 13:23:58 +0000

Roll-back r250822.Summary: It breaks the build for the ASTMatchersSubscribers: klimek, cfe-commitsDifferential Revision: http://reviews.llvm.org/D13893llvm-svn: 250827 List of files: /llvm-project-15.0.7/clang/lib/Frontend/Rewrite/FixItRewriter.cpp

b5250d34 - Apply modernize-use-default to clang.

Tue, 20 Oct 2015 12:52:55 +0000

Apply modernize-use-default to clang.Summary: Replace empty bodies of default constructors and destructors with '= default'.Reviewers: bkramer, klimekSubscribers: klimek, alexfh, cfe-commitsDifferential Revision: http://reviews.llvm.org/D13890llvm-svn: 250822 List of files: /llvm-project-15.0.7/clang/lib/Frontend/Rewrite/FixItRewriter.cpp

ab9db510 - Revert r240270 ("Fixed/added namespace ending comments using clang-tidy").

Mon, 22 Jun 2015 23:07:51 +0000

Revert r240270 ("Fixed/added namespace ending comments using clang-tidy").llvm-svn: 240353 List of files: /llvm-project-15.0.7/clang/lib/Frontend/Rewrite/FixItRewriter.cpp

3d9d929e - Fixed/added namespace ending comments using clang-tidy. NFC

Mon, 22 Jun 2015 09:47:44 +0000

Fixed/added namespace ending comments using clang-tidy. NFCThe patch is generated using this command: $ tools/extra/clang-tidy/tool/run-clang-tidy.py -fix \ -checks=-*,llvm-namespace-comment -header-filter='llvm/.*|clang/.*' \ work/llvm/tools/clangTo reduce churn, not touching namespaces spanning less than 10 lines.llvm-svn: 240270 List of files: /llvm-project-15.0.7/clang/lib/Frontend/Rewrite/FixItRewriter.cpp

3df5dd4d - [fixit] Use overwriteChangedFiles() to deal with Windows mapped files

Wed, 17 Jun 2015 17:47:30 +0000

[fixit] Use overwriteChangedFiles() to deal with Windows mapped filesFixes one instance of PR17960.llvm-svn: 239920 List of files: /llvm-project-15.0.7/clang/lib/Frontend/Rewrite/FixItRewriter.cpp

41c247a6 - Make DiagnosticsEngine::takeClient return std::unique_ptr<>

Mon, 17 Nov 2014 23:46:02 +0000

Make DiagnosticsEngine::takeClient return std::unique_ptr<>Summary:Make DiagnosticsEngine::takeClient return std::unique_ptr<>. Updatedcallers to store conditional ownership using a pair of pointer and unique_ptrinstead of a pointer + bool. Updated code that temporarily registers clients touse the non-owning registration (+ removed extra calls to takeClient).Reviewers: dblaikieReviewed By: dblaikieSubscribers: cfe-commitsDifferential Revision: http://reviews.llvm.org/D6294llvm-svn: 222193 List of files: /llvm-project-15.0.7/clang/lib/Frontend/Rewrite/FixItRewriter.cpp

dae941a6 - Update for llvm api change.

Mon, 25 Aug 2014 18:17:04 +0000

Update for llvm api change.llvm-svn: 216397 List of files: /llvm-project-15.0.7/clang/lib/Frontend/Rewrite/FixItRewriter.cpp

0621cb2e - Make clang's rewrite engine a core feature

Wed, 16 Jul 2014 16:48:33 +0000

Make clang's rewrite engine a core featureThe rewrite facility's footprint is small so it's not worth going to theselengths to support disabling at configure time, particularly since key compilerfeatures now depend on it.Meanwhile the Objective-C rewriters have been moved under theENABLE_CLANG_ARCMT umbrella for now as they're comparatively heavy and stillpotentially worth excluding from lightweight builds.Tests are now passing with any combination of feature flags. The flagshistorically haven't been tested by LLVM's build servers so caveat emptor.llvm-svn: 213171 List of files: /llvm-project-15.0.7/clang/lib/Frontend/Rewrite/FixItRewriter.cpp