================
@@ -24,7 +24,7 @@ namespace clang::tidy {
 /// them in the order of appearance in the list.
 class GlobList {
 public:
-  virtual ~GlobList() = default;
+  GlobList() = default;
----------------
localspook wrote:

Yeah, if downstream users depend on `GlobList` being polymorphic, I can see how 
this could be a nasty silent break for them. In that case, closing.

https://github.com/llvm/llvm-project/pull/164212
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to