summaryrefslogtreecommitdiffstats
path: root/yql/essentials/udfs/common/python/python_udf/python_udf.cpp
diff options
context:
space:
mode:
authorvitya-smirnov <[email protected]>2026-01-13 17:28:38 +0300
committervitya-smirnov <[email protected]>2026-01-13 17:55:02 +0300
commitdcda00624e675c16afa086ce13d8f583f7929846 (patch)
treef2b2c1b62e8b8989719b4a417b1e9a5480b368a8 /yql/essentials/udfs/common/python/python_udf/python_udf.cpp
parent1c54c418a34fa23196495b69afad0abdda4ee6bd (diff)
YQL-20095: Enable modernize-use-override
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
Diffstat (limited to 'yql/essentials/udfs/common/python/python_udf/python_udf.cpp')
-rw-r--r--yql/essentials/udfs/common/python/python_udf/python_udf.cpp2
1 files changed, 1 insertions, 1 deletions
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();