From dcda00624e675c16afa086ce13d8f583f7929846 Mon Sep 17 00:00:00 2001 From: vitya-smirnov Date: Tue, 13 Jan 2026 17:28:38 +0300 Subject: YQL-20095: Enable modernize-use-override MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The check `modernize-use-override` is useful as it: 1. Removes redundant `override` when it is used with a `final`. 2. Removes redundant `virtual` or replace it with a `override`. So it is more clean for a reader, that overriding happens. It is enabled not out of order, as it is just an alias to `cppcoreguidelines-explicit-virtual-functions`. I also decided to switch a strategy of enabling checks. Now I will enable only a single rule with a single PR and prefer rules with a non-breaking autofix. In the new year with new linter checks! 🎄 commit_hash:e6e233baa90b31e5f65e11837546690c47f71ab5 --- yql/essentials/udfs/common/python/python_udf/python_udf.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'yql/essentials/udfs/common/python/python_udf/python_udf.cpp') diff --git a/yql/essentials/udfs/common/python/python_udf/python_udf.cpp b/yql/essentials/udfs/common/python/python_udf/python_udf.cpp index fd0743962ba..a70e69a3257 100644 --- a/yql/essentials/udfs/common/python/python_udf/python_udf.cpp +++ b/yql/essentials/udfs/common/python/python_udf/python_udf.cpp @@ -90,7 +90,7 @@ public: } } - ~TPythonModule() { + ~TPythonModule() override { if (Standalone_) { PyEval_RestoreThread(MainThreadState_); Py_Finalize(); -- cgit v1.3