From f73593a4034d9eec0ec4466b8e173d4a4daece1f Mon Sep 17 00:00:00 2001 From: David Kalnischkies Date: Thu, 26 Dec 2024 19:41:53 +0000 Subject: Drop usage of macro APT_OVERRIDE for simple override We were rather inconsistent in using it and as our public headers contain deduction guides (a c++17 feature) it seems silly to try to hide a c++11 feature in a macro, so lets stop this charade and drop the macro and while we are changing all these lines lets apply [[nodiscard]] (another c++17 feature) and other suggestions from clang-tidy and formatting for a little more consistency. --- test/libapt/acqprogress_test.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'test/libapt/acqprogress_test.cc') diff --git a/test/libapt/acqprogress_test.cc b/test/libapt/acqprogress_test.cc index 17a04161e..16b17c3a4 100644 --- a/test/libapt/acqprogress_test.cc +++ b/test/libapt/acqprogress_test.cc @@ -13,8 +13,8 @@ class TestItem: public pkgAcquire::Item public: explicit TestItem(pkgAcquire * const Acq) : pkgAcquire::Item(Acq) {} - virtual std::string DescURI() const APT_OVERRIDE { return ""; } - virtual HashStringList GetExpectedHashes() const APT_OVERRIDE { return HashStringList(); } + [[nodiscard]] std::string DescURI() const override { return ""; } + [[nodiscard]] HashStringList GetExpectedHashes() const override { return {}; } }; -- cgit v1.2.3-70-g09d2