diff options
| author | vitya-smirnov <[email protected]> | 2026-01-13 17:28:38 +0300 |
|---|---|---|
| committer | vitya-smirnov <[email protected]> | 2026-01-13 17:55:02 +0300 |
| commit | dcda00624e675c16afa086ce13d8f583f7929846 (patch) | |
| tree | f2b2c1b62e8b8989719b4a417b1e9a5480b368a8 /yql/essentials/udfs/common/python/python_udf/python_udf.cpp | |
| parent | 1c54c418a34fa23196495b69afad0abdda4ee6bd (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.cpp | 2 |
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(); |
