1e5dd7070Spatrick //===--- AtomicChange.cpp - AtomicChange implementation -----------------*- C++ -*-===//
2e5dd7070Spatrick //
3e5dd7070Spatrick // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
4e5dd7070Spatrick // See https://llvm.org/LICENSE.txt for license information.
5e5dd7070Spatrick // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
6e5dd7070Spatrick //
7e5dd7070Spatrick //===----------------------------------------------------------------------===//
8e5dd7070Spatrick
9e5dd7070Spatrick #include "clang/Tooling/Refactoring/AtomicChange.h"
10e5dd7070Spatrick #include "clang/Tooling/ReplacementsYaml.h"
11e5dd7070Spatrick #include "llvm/Support/YAMLTraits.h"
12e5dd7070Spatrick #include <string>
13e5dd7070Spatrick
14e5dd7070Spatrick LLVM_YAML_IS_SEQUENCE_VECTOR(clang::tooling::AtomicChange)
15e5dd7070Spatrick
16e5dd7070Spatrick namespace {
17e5dd7070Spatrick /// Helper to (de)serialize an AtomicChange since we don't have direct
18e5dd7070Spatrick /// access to its data members.
19e5dd7070Spatrick /// Data members of a normalized AtomicChange can be directly mapped from/to
20e5dd7070Spatrick /// YAML string.
21e5dd7070Spatrick struct NormalizedAtomicChange {
22e5dd7070Spatrick NormalizedAtomicChange() = default;
23e5dd7070Spatrick
NormalizedAtomicChange__anond7eb80c20111::NormalizedAtomicChange24e5dd7070Spatrick NormalizedAtomicChange(const llvm::yaml::IO &) {}
25e5dd7070Spatrick
26e5dd7070Spatrick // This converts AtomicChange's internal implementation of the replacements
27e5dd7070Spatrick // set to a vector of replacements.
NormalizedAtomicChange__anond7eb80c20111::NormalizedAtomicChange28e5dd7070Spatrick NormalizedAtomicChange(const llvm::yaml::IO &,
29e5dd7070Spatrick const clang::tooling::AtomicChange &E)
30e5dd7070Spatrick : Key(E.getKey()), FilePath(E.getFilePath()), Error(E.getError()),
31e5dd7070Spatrick InsertedHeaders(E.getInsertedHeaders()),
32e5dd7070Spatrick RemovedHeaders(E.getRemovedHeaders()),
33e5dd7070Spatrick Replaces(E.getReplacements().begin(), E.getReplacements().end()) {}
34e5dd7070Spatrick
35e5dd7070Spatrick // This is not expected to be called but needed for template instantiation.
denormalize__anond7eb80c20111::NormalizedAtomicChange36e5dd7070Spatrick clang::tooling::AtomicChange denormalize(const llvm::yaml::IO &) {
37e5dd7070Spatrick llvm_unreachable("Do not convert YAML to AtomicChange directly with '>>'. "
38e5dd7070Spatrick "Use AtomicChange::convertFromYAML instead.");
39e5dd7070Spatrick }
40e5dd7070Spatrick std::string Key;
41e5dd7070Spatrick std::string FilePath;
42e5dd7070Spatrick std::string Error;
43e5dd7070Spatrick std::vector<std::string> InsertedHeaders;
44e5dd7070Spatrick std::vector<std::string> RemovedHeaders;
45e5dd7070Spatrick std::vector<clang::tooling::Replacement> Replaces;
46e5dd7070Spatrick };
47e5dd7070Spatrick } // anonymous namespace
48e5dd7070Spatrick
49e5dd7070Spatrick namespace llvm {
50e5dd7070Spatrick namespace yaml {
51e5dd7070Spatrick
52e5dd7070Spatrick /// Specialized MappingTraits to describe how an AtomicChange is
53e5dd7070Spatrick /// (de)serialized.
54e5dd7070Spatrick template <> struct MappingTraits<NormalizedAtomicChange> {
mappingllvm::yaml::MappingTraits55e5dd7070Spatrick static void mapping(IO &Io, NormalizedAtomicChange &Doc) {
56e5dd7070Spatrick Io.mapRequired("Key", Doc.Key);
57e5dd7070Spatrick Io.mapRequired("FilePath", Doc.FilePath);
58e5dd7070Spatrick Io.mapRequired("Error", Doc.Error);
59e5dd7070Spatrick Io.mapRequired("InsertedHeaders", Doc.InsertedHeaders);
60e5dd7070Spatrick Io.mapRequired("RemovedHeaders", Doc.RemovedHeaders);
61e5dd7070Spatrick Io.mapRequired("Replacements", Doc.Replaces);
62e5dd7070Spatrick }
63e5dd7070Spatrick };
64e5dd7070Spatrick
65e5dd7070Spatrick /// Specialized MappingTraits to describe how an AtomicChange is
66e5dd7070Spatrick /// (de)serialized.
67e5dd7070Spatrick template <> struct MappingTraits<clang::tooling::AtomicChange> {
mappingllvm::yaml::MappingTraits68e5dd7070Spatrick static void mapping(IO &Io, clang::tooling::AtomicChange &Doc) {
69e5dd7070Spatrick MappingNormalization<NormalizedAtomicChange, clang::tooling::AtomicChange>
70e5dd7070Spatrick Keys(Io, Doc);
71e5dd7070Spatrick Io.mapRequired("Key", Keys->Key);
72e5dd7070Spatrick Io.mapRequired("FilePath", Keys->FilePath);
73e5dd7070Spatrick Io.mapRequired("Error", Keys->Error);
74e5dd7070Spatrick Io.mapRequired("InsertedHeaders", Keys->InsertedHeaders);
75e5dd7070Spatrick Io.mapRequired("RemovedHeaders", Keys->RemovedHeaders);
76e5dd7070Spatrick Io.mapRequired("Replacements", Keys->Replaces);
77e5dd7070Spatrick }
78e5dd7070Spatrick };
79e5dd7070Spatrick
80e5dd7070Spatrick } // end namespace yaml
81e5dd7070Spatrick } // end namespace llvm
82e5dd7070Spatrick
83e5dd7070Spatrick namespace clang {
84e5dd7070Spatrick namespace tooling {
85e5dd7070Spatrick namespace {
86e5dd7070Spatrick
87e5dd7070Spatrick // Returns true if there is any line that violates \p ColumnLimit in range
88e5dd7070Spatrick // [Start, End].
violatesColumnLimit(llvm::StringRef Code,unsigned ColumnLimit,unsigned Start,unsigned End)89e5dd7070Spatrick bool violatesColumnLimit(llvm::StringRef Code, unsigned ColumnLimit,
90e5dd7070Spatrick unsigned Start, unsigned End) {
91e5dd7070Spatrick auto StartPos = Code.rfind('\n', Start);
92e5dd7070Spatrick StartPos = (StartPos == llvm::StringRef::npos) ? 0 : StartPos + 1;
93e5dd7070Spatrick
94e5dd7070Spatrick auto EndPos = Code.find("\n", End);
95e5dd7070Spatrick if (EndPos == llvm::StringRef::npos)
96e5dd7070Spatrick EndPos = Code.size();
97e5dd7070Spatrick
98e5dd7070Spatrick llvm::SmallVector<llvm::StringRef, 8> Lines;
99e5dd7070Spatrick Code.substr(StartPos, EndPos - StartPos).split(Lines, '\n');
100e5dd7070Spatrick for (llvm::StringRef Line : Lines)
101e5dd7070Spatrick if (Line.size() > ColumnLimit)
102e5dd7070Spatrick return true;
103e5dd7070Spatrick return false;
104e5dd7070Spatrick }
105e5dd7070Spatrick
106e5dd7070Spatrick std::vector<Range>
getRangesForFormating(llvm::StringRef Code,unsigned ColumnLimit,ApplyChangesSpec::FormatOption Format,const clang::tooling::Replacements & Replaces)107e5dd7070Spatrick getRangesForFormating(llvm::StringRef Code, unsigned ColumnLimit,
108e5dd7070Spatrick ApplyChangesSpec::FormatOption Format,
109e5dd7070Spatrick const clang::tooling::Replacements &Replaces) {
110e5dd7070Spatrick // kNone suppresses formatting entirely.
111e5dd7070Spatrick if (Format == ApplyChangesSpec::kNone)
112e5dd7070Spatrick return {};
113e5dd7070Spatrick std::vector<clang::tooling::Range> Ranges;
114e5dd7070Spatrick // This works assuming that replacements are ordered by offset.
115e5dd7070Spatrick // FIXME: use `getAffectedRanges()` to calculate when it does not include '\n'
116e5dd7070Spatrick // at the end of an insertion in affected ranges.
117e5dd7070Spatrick int Offset = 0;
118e5dd7070Spatrick for (const clang::tooling::Replacement &R : Replaces) {
119e5dd7070Spatrick int Start = R.getOffset() + Offset;
120e5dd7070Spatrick int End = Start + R.getReplacementText().size();
121e5dd7070Spatrick if (!R.getReplacementText().empty() &&
122e5dd7070Spatrick R.getReplacementText().back() == '\n' && R.getLength() == 0 &&
123e5dd7070Spatrick R.getOffset() > 0 && R.getOffset() <= Code.size() &&
124e5dd7070Spatrick Code[R.getOffset() - 1] == '\n')
125e5dd7070Spatrick // If we are inserting at the start of a line and the replacement ends in
126e5dd7070Spatrick // a newline, we don't need to format the subsequent line.
127e5dd7070Spatrick --End;
128e5dd7070Spatrick Offset += R.getReplacementText().size() - R.getLength();
129e5dd7070Spatrick
130e5dd7070Spatrick if (Format == ApplyChangesSpec::kAll ||
131e5dd7070Spatrick violatesColumnLimit(Code, ColumnLimit, Start, End))
132e5dd7070Spatrick Ranges.emplace_back(Start, End - Start);
133e5dd7070Spatrick }
134e5dd7070Spatrick return Ranges;
135e5dd7070Spatrick }
136e5dd7070Spatrick
make_string_error(const llvm::Twine & Message)137e5dd7070Spatrick inline llvm::Error make_string_error(const llvm::Twine &Message) {
138e5dd7070Spatrick return llvm::make_error<llvm::StringError>(Message,
139e5dd7070Spatrick llvm::inconvertibleErrorCode());
140e5dd7070Spatrick }
141e5dd7070Spatrick
142e5dd7070Spatrick // Creates replacements for inserting/deleting #include headers.
143e5dd7070Spatrick llvm::Expected<Replacements>
createReplacementsForHeaders(llvm::StringRef FilePath,llvm::StringRef Code,llvm::ArrayRef<AtomicChange> Changes,const format::FormatStyle & Style)144e5dd7070Spatrick createReplacementsForHeaders(llvm::StringRef FilePath, llvm::StringRef Code,
145e5dd7070Spatrick llvm::ArrayRef<AtomicChange> Changes,
146e5dd7070Spatrick const format::FormatStyle &Style) {
147e5dd7070Spatrick // Create header insertion/deletion replacements to be cleaned up
148e5dd7070Spatrick // (i.e. converted to real insertion/deletion replacements).
149e5dd7070Spatrick Replacements HeaderReplacements;
150e5dd7070Spatrick for (const auto &Change : Changes) {
151e5dd7070Spatrick for (llvm::StringRef Header : Change.getInsertedHeaders()) {
152e5dd7070Spatrick std::string EscapedHeader =
153e5dd7070Spatrick Header.startswith("<") || Header.startswith("\"")
154e5dd7070Spatrick ? Header.str()
155e5dd7070Spatrick : ("\"" + Header + "\"").str();
156e5dd7070Spatrick std::string ReplacementText = "#include " + EscapedHeader;
157e5dd7070Spatrick // Offset UINT_MAX and length 0 indicate that the replacement is a header
158e5dd7070Spatrick // insertion.
159e5dd7070Spatrick llvm::Error Err = HeaderReplacements.add(
160e5dd7070Spatrick tooling::Replacement(FilePath, UINT_MAX, 0, ReplacementText));
161e5dd7070Spatrick if (Err)
162e5dd7070Spatrick return std::move(Err);
163e5dd7070Spatrick }
164e5dd7070Spatrick for (const std::string &Header : Change.getRemovedHeaders()) {
165e5dd7070Spatrick // Offset UINT_MAX and length 1 indicate that the replacement is a header
166e5dd7070Spatrick // deletion.
167e5dd7070Spatrick llvm::Error Err =
168e5dd7070Spatrick HeaderReplacements.add(Replacement(FilePath, UINT_MAX, 1, Header));
169e5dd7070Spatrick if (Err)
170e5dd7070Spatrick return std::move(Err);
171e5dd7070Spatrick }
172e5dd7070Spatrick }
173e5dd7070Spatrick
174e5dd7070Spatrick // cleanupAroundReplacements() converts header insertions/deletions into
175e5dd7070Spatrick // actual replacements that add/remove headers at the right location.
176e5dd7070Spatrick return clang::format::cleanupAroundReplacements(Code, HeaderReplacements,
177e5dd7070Spatrick Style);
178e5dd7070Spatrick }
179e5dd7070Spatrick
180e5dd7070Spatrick // Combine replacements in all Changes as a `Replacements`. This ignores the
181e5dd7070Spatrick // file path in all replacements and replaces them with \p FilePath.
182e5dd7070Spatrick llvm::Expected<Replacements>
combineReplacementsInChanges(llvm::StringRef FilePath,llvm::ArrayRef<AtomicChange> Changes)183e5dd7070Spatrick combineReplacementsInChanges(llvm::StringRef FilePath,
184e5dd7070Spatrick llvm::ArrayRef<AtomicChange> Changes) {
185e5dd7070Spatrick Replacements Replaces;
186e5dd7070Spatrick for (const auto &Change : Changes)
187e5dd7070Spatrick for (const auto &R : Change.getReplacements())
188e5dd7070Spatrick if (auto Err = Replaces.add(Replacement(
189e5dd7070Spatrick FilePath, R.getOffset(), R.getLength(), R.getReplacementText())))
190e5dd7070Spatrick return std::move(Err);
191e5dd7070Spatrick return Replaces;
192e5dd7070Spatrick }
193e5dd7070Spatrick
194e5dd7070Spatrick } // end namespace
195e5dd7070Spatrick
AtomicChange(const SourceManager & SM,SourceLocation KeyPosition)196e5dd7070Spatrick AtomicChange::AtomicChange(const SourceManager &SM,
197e5dd7070Spatrick SourceLocation KeyPosition) {
198e5dd7070Spatrick const FullSourceLoc FullKeyPosition(KeyPosition, SM);
199e5dd7070Spatrick std::pair<FileID, unsigned> FileIDAndOffset =
200e5dd7070Spatrick FullKeyPosition.getSpellingLoc().getDecomposedLoc();
201e5dd7070Spatrick const FileEntry *FE = SM.getFileEntryForID(FileIDAndOffset.first);
202e5dd7070Spatrick assert(FE && "Cannot create AtomicChange with invalid location.");
203*ec727ea7Spatrick FilePath = std::string(FE->getName());
204e5dd7070Spatrick Key = FilePath + ":" + std::to_string(FileIDAndOffset.second);
205e5dd7070Spatrick }
206e5dd7070Spatrick
AtomicChange(const SourceManager & SM,SourceLocation KeyPosition,llvm::Any M)207*ec727ea7Spatrick AtomicChange::AtomicChange(const SourceManager &SM, SourceLocation KeyPosition,
208*ec727ea7Spatrick llvm::Any M)
209*ec727ea7Spatrick : AtomicChange(SM, KeyPosition) {
210*ec727ea7Spatrick Metadata = std::move(M);
211*ec727ea7Spatrick }
212*ec727ea7Spatrick
AtomicChange(std::string Key,std::string FilePath,std::string Error,std::vector<std::string> InsertedHeaders,std::vector<std::string> RemovedHeaders,clang::tooling::Replacements Replaces)213e5dd7070Spatrick AtomicChange::AtomicChange(std::string Key, std::string FilePath,
214e5dd7070Spatrick std::string Error,
215e5dd7070Spatrick std::vector<std::string> InsertedHeaders,
216e5dd7070Spatrick std::vector<std::string> RemovedHeaders,
217e5dd7070Spatrick clang::tooling::Replacements Replaces)
218e5dd7070Spatrick : Key(std::move(Key)), FilePath(std::move(FilePath)),
219e5dd7070Spatrick Error(std::move(Error)), InsertedHeaders(std::move(InsertedHeaders)),
220e5dd7070Spatrick RemovedHeaders(std::move(RemovedHeaders)), Replaces(std::move(Replaces)) {
221e5dd7070Spatrick }
222e5dd7070Spatrick
operator ==(const AtomicChange & Other) const223e5dd7070Spatrick bool AtomicChange::operator==(const AtomicChange &Other) const {
224e5dd7070Spatrick if (Key != Other.Key || FilePath != Other.FilePath || Error != Other.Error)
225e5dd7070Spatrick return false;
226e5dd7070Spatrick if (!(Replaces == Other.Replaces))
227e5dd7070Spatrick return false;
228e5dd7070Spatrick // FXIME: Compare header insertions/removals.
229e5dd7070Spatrick return true;
230e5dd7070Spatrick }
231e5dd7070Spatrick
toYAMLString()232e5dd7070Spatrick std::string AtomicChange::toYAMLString() {
233e5dd7070Spatrick std::string YamlContent;
234e5dd7070Spatrick llvm::raw_string_ostream YamlContentStream(YamlContent);
235e5dd7070Spatrick
236e5dd7070Spatrick llvm::yaml::Output YAML(YamlContentStream);
237e5dd7070Spatrick YAML << *this;
238e5dd7070Spatrick YamlContentStream.flush();
239e5dd7070Spatrick return YamlContent;
240e5dd7070Spatrick }
241e5dd7070Spatrick
convertFromYAML(llvm::StringRef YAMLContent)242e5dd7070Spatrick AtomicChange AtomicChange::convertFromYAML(llvm::StringRef YAMLContent) {
243e5dd7070Spatrick NormalizedAtomicChange NE;
244e5dd7070Spatrick llvm::yaml::Input YAML(YAMLContent);
245e5dd7070Spatrick YAML >> NE;
246e5dd7070Spatrick AtomicChange E(NE.Key, NE.FilePath, NE.Error, NE.InsertedHeaders,
247e5dd7070Spatrick NE.RemovedHeaders, tooling::Replacements());
248e5dd7070Spatrick for (const auto &R : NE.Replaces) {
249e5dd7070Spatrick llvm::Error Err = E.Replaces.add(R);
250e5dd7070Spatrick if (Err)
251e5dd7070Spatrick llvm_unreachable(
252e5dd7070Spatrick "Failed to add replacement when Converting YAML to AtomicChange.");
253e5dd7070Spatrick llvm::consumeError(std::move(Err));
254e5dd7070Spatrick }
255e5dd7070Spatrick return E;
256e5dd7070Spatrick }
257e5dd7070Spatrick
replace(const SourceManager & SM,const CharSourceRange & Range,llvm::StringRef ReplacementText)258e5dd7070Spatrick llvm::Error AtomicChange::replace(const SourceManager &SM,
259e5dd7070Spatrick const CharSourceRange &Range,
260e5dd7070Spatrick llvm::StringRef ReplacementText) {
261e5dd7070Spatrick return Replaces.add(Replacement(SM, Range, ReplacementText));
262e5dd7070Spatrick }
263e5dd7070Spatrick
replace(const SourceManager & SM,SourceLocation Loc,unsigned Length,llvm::StringRef Text)264e5dd7070Spatrick llvm::Error AtomicChange::replace(const SourceManager &SM, SourceLocation Loc,
265e5dd7070Spatrick unsigned Length, llvm::StringRef Text) {
266e5dd7070Spatrick return Replaces.add(Replacement(SM, Loc, Length, Text));
267e5dd7070Spatrick }
268e5dd7070Spatrick
insert(const SourceManager & SM,SourceLocation Loc,llvm::StringRef Text,bool InsertAfter)269e5dd7070Spatrick llvm::Error AtomicChange::insert(const SourceManager &SM, SourceLocation Loc,
270e5dd7070Spatrick llvm::StringRef Text, bool InsertAfter) {
271e5dd7070Spatrick if (Text.empty())
272e5dd7070Spatrick return llvm::Error::success();
273e5dd7070Spatrick Replacement R(SM, Loc, 0, Text);
274e5dd7070Spatrick llvm::Error Err = Replaces.add(R);
275e5dd7070Spatrick if (Err) {
276e5dd7070Spatrick return llvm::handleErrors(
277e5dd7070Spatrick std::move(Err), [&](const ReplacementError &RE) -> llvm::Error {
278e5dd7070Spatrick if (RE.get() != replacement_error::insert_conflict)
279e5dd7070Spatrick return llvm::make_error<ReplacementError>(RE);
280e5dd7070Spatrick unsigned NewOffset = Replaces.getShiftedCodePosition(R.getOffset());
281e5dd7070Spatrick if (!InsertAfter)
282e5dd7070Spatrick NewOffset -=
283e5dd7070Spatrick RE.getExistingReplacement()->getReplacementText().size();
284e5dd7070Spatrick Replacement NewR(R.getFilePath(), NewOffset, 0, Text);
285e5dd7070Spatrick Replaces = Replaces.merge(Replacements(NewR));
286e5dd7070Spatrick return llvm::Error::success();
287e5dd7070Spatrick });
288e5dd7070Spatrick }
289e5dd7070Spatrick return llvm::Error::success();
290e5dd7070Spatrick }
291e5dd7070Spatrick
addHeader(llvm::StringRef Header)292e5dd7070Spatrick void AtomicChange::addHeader(llvm::StringRef Header) {
293*ec727ea7Spatrick InsertedHeaders.push_back(std::string(Header));
294e5dd7070Spatrick }
295e5dd7070Spatrick
removeHeader(llvm::StringRef Header)296e5dd7070Spatrick void AtomicChange::removeHeader(llvm::StringRef Header) {
297*ec727ea7Spatrick RemovedHeaders.push_back(std::string(Header));
298e5dd7070Spatrick }
299e5dd7070Spatrick
300e5dd7070Spatrick llvm::Expected<std::string>
applyAtomicChanges(llvm::StringRef FilePath,llvm::StringRef Code,llvm::ArrayRef<AtomicChange> Changes,const ApplyChangesSpec & Spec)301e5dd7070Spatrick applyAtomicChanges(llvm::StringRef FilePath, llvm::StringRef Code,
302e5dd7070Spatrick llvm::ArrayRef<AtomicChange> Changes,
303e5dd7070Spatrick const ApplyChangesSpec &Spec) {
304e5dd7070Spatrick llvm::Expected<Replacements> HeaderReplacements =
305e5dd7070Spatrick createReplacementsForHeaders(FilePath, Code, Changes, Spec.Style);
306e5dd7070Spatrick if (!HeaderReplacements)
307e5dd7070Spatrick return make_string_error(
308e5dd7070Spatrick "Failed to create replacements for header changes: " +
309e5dd7070Spatrick llvm::toString(HeaderReplacements.takeError()));
310e5dd7070Spatrick
311e5dd7070Spatrick llvm::Expected<Replacements> Replaces =
312e5dd7070Spatrick combineReplacementsInChanges(FilePath, Changes);
313e5dd7070Spatrick if (!Replaces)
314e5dd7070Spatrick return make_string_error("Failed to combine replacements in all changes: " +
315e5dd7070Spatrick llvm::toString(Replaces.takeError()));
316e5dd7070Spatrick
317e5dd7070Spatrick Replacements AllReplaces = std::move(*Replaces);
318e5dd7070Spatrick for (const auto &R : *HeaderReplacements) {
319e5dd7070Spatrick llvm::Error Err = AllReplaces.add(R);
320e5dd7070Spatrick if (Err)
321e5dd7070Spatrick return make_string_error(
322e5dd7070Spatrick "Failed to combine existing replacements with header replacements: " +
323e5dd7070Spatrick llvm::toString(std::move(Err)));
324e5dd7070Spatrick }
325e5dd7070Spatrick
326e5dd7070Spatrick if (Spec.Cleanup) {
327e5dd7070Spatrick llvm::Expected<Replacements> CleanReplaces =
328e5dd7070Spatrick format::cleanupAroundReplacements(Code, AllReplaces, Spec.Style);
329e5dd7070Spatrick if (!CleanReplaces)
330e5dd7070Spatrick return make_string_error("Failed to cleanup around replacements: " +
331e5dd7070Spatrick llvm::toString(CleanReplaces.takeError()));
332e5dd7070Spatrick AllReplaces = std::move(*CleanReplaces);
333e5dd7070Spatrick }
334e5dd7070Spatrick
335e5dd7070Spatrick // Apply all replacements.
336e5dd7070Spatrick llvm::Expected<std::string> ChangedCode =
337e5dd7070Spatrick applyAllReplacements(Code, AllReplaces);
338e5dd7070Spatrick if (!ChangedCode)
339e5dd7070Spatrick return make_string_error("Failed to apply all replacements: " +
340e5dd7070Spatrick llvm::toString(ChangedCode.takeError()));
341e5dd7070Spatrick
342e5dd7070Spatrick // Sort inserted headers. This is done even if other formatting is turned off
343e5dd7070Spatrick // as incorrectly sorted headers are always just wrong, it's not a matter of
344e5dd7070Spatrick // taste.
345e5dd7070Spatrick Replacements HeaderSortingReplacements = format::sortIncludes(
346e5dd7070Spatrick Spec.Style, *ChangedCode, AllReplaces.getAffectedRanges(), FilePath);
347e5dd7070Spatrick ChangedCode = applyAllReplacements(*ChangedCode, HeaderSortingReplacements);
348e5dd7070Spatrick if (!ChangedCode)
349e5dd7070Spatrick return make_string_error(
350e5dd7070Spatrick "Failed to apply replacements for sorting includes: " +
351e5dd7070Spatrick llvm::toString(ChangedCode.takeError()));
352e5dd7070Spatrick
353e5dd7070Spatrick AllReplaces = AllReplaces.merge(HeaderSortingReplacements);
354e5dd7070Spatrick
355e5dd7070Spatrick std::vector<Range> FormatRanges = getRangesForFormating(
356e5dd7070Spatrick *ChangedCode, Spec.Style.ColumnLimit, Spec.Format, AllReplaces);
357e5dd7070Spatrick if (!FormatRanges.empty()) {
358e5dd7070Spatrick Replacements FormatReplacements =
359e5dd7070Spatrick format::reformat(Spec.Style, *ChangedCode, FormatRanges, FilePath);
360e5dd7070Spatrick ChangedCode = applyAllReplacements(*ChangedCode, FormatReplacements);
361e5dd7070Spatrick if (!ChangedCode)
362e5dd7070Spatrick return make_string_error(
363e5dd7070Spatrick "Failed to apply replacements for formatting changed code: " +
364e5dd7070Spatrick llvm::toString(ChangedCode.takeError()));
365e5dd7070Spatrick }
366e5dd7070Spatrick return ChangedCode;
367e5dd7070Spatrick }
368e5dd7070Spatrick
369e5dd7070Spatrick } // end namespace tooling
370e5dd7070Spatrick } // end namespace clang
371