ymandel updated this revision to Diff 227457.
ymandel added a comment.
renamed free function to avoid overloading a templated function.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69632/new/
https://reviews.llvm.org/D69632
Files:
clang/include/clang/Tooling/Transformer/Stencil.h
Index: clang/include/clang/Tooling/Transformer/Stencil.h
===================================================================
--- clang/include/clang/Tooling/Transformer/Stencil.h
+++ clang/include/clang/Tooling/Transformer/Stencil.h
@@ -90,6 +90,7 @@
class Stencil {
public:
Stencil() = default;
+ Stencil(std::vector<StencilPart> Parts) : Parts(std::move(Parts)) {}
/// Composes a stencil from a series of parts.
template <typename... Ts> static Stencil cat(Ts &&... Parts) {
@@ -140,6 +141,12 @@
template <typename... Ts> Stencil cat(Ts &&... Parts) {
return Stencil::cat(std::forward<Ts>(Parts)...);
}
+/// Convenience wrapper for Stencil constructor of the same type. Declaration
+/// outside of the class supports transition of `Stencil` type to an alias
+/// rather than a class.
+inline Stencil catVector(std::vector<StencilPart> Parts) {
+ return Stencil(std::move(Parts));
+}
/// \returns exactly the text provided.
StencilPart text(llvm::StringRef Text);
Index: clang/include/clang/Tooling/Transformer/Stencil.h
===================================================================
--- clang/include/clang/Tooling/Transformer/Stencil.h
+++ clang/include/clang/Tooling/Transformer/Stencil.h
@@ -90,6 +90,7 @@
class Stencil {
public:
Stencil() = default;
+ Stencil(std::vector<StencilPart> Parts) : Parts(std::move(Parts)) {}
/// Composes a stencil from a series of parts.
template <typename... Ts> static Stencil cat(Ts &&... Parts) {
@@ -140,6 +141,12 @@
template <typename... Ts> Stencil cat(Ts &&... Parts) {
return Stencil::cat(std::forward<Ts>(Parts)...);
}
+/// Convenience wrapper for Stencil constructor of the same type. Declaration
+/// outside of the class supports transition of `Stencil` type to an alias
+/// rather than a class.
+inline Stencil catVector(std::vector<StencilPart> Parts) {
+ return Stencil(std::move(Parts));
+}
/// \returns exactly the text provided.
StencilPart text(llvm::StringRef Text);
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
