Merge branch 'fix/gccwarnings' into 'main'

Build with and fix warnings of gcc-13

See merge request apt-team/apt!280

4 files changed, 38 insertions, 34 deletions

diff --git a/apt-pkg/acquire.cc b/apt-pkg/acquire.cc
index dd0624ad6..cdb2c5257 100644
--- a/apt-pkg/acquire.cc
+++ b/apt-pkg/acquire.cc
@@ -1470,7 +1470,8 @@ bool pkgAcquireStatus::Pulse(pkgAcquire *Owner)
       std::string msg;
       long i = CurrentItems < TotalItems ? CurrentItems + 1 : CurrentItems;
       // only show the ETA if it makes sense
-      if (ETA > 0 && ETA < std::chrono::seconds(std::chrono::hours(24 * 2)).count())
+      auto const twodays = std::chrono::seconds(std::chrono::hours(24 * 2)).count();
+      if (ETA > 0 && ETA < static_cast<decltype(ETA)>(twodays))
 	 strprintf(msg, _("Retrieving file %li of %li (%s remaining)"), i, TotalItems, TimeToStr(ETA).c_str());
       else
 	 strprintf(msg, _("Retrieving file %li of %li"), i, TotalItems);
diff --git a/apt-pkg/cachefilter.h b/apt-pkg/cachefilter.h
index ccff6f9a3..1b5f9aa71 100644
--- a/apt-pkg/cachefilter.h
+++ b/apt-pkg/cachefilter.h
@@ -30,35 +30,35 @@ public:
 
 class APT_PUBLIC PackageMatcher : public Matcher {
 public:
-   virtual bool operator() (pkgCache::PkgIterator const &Pkg) APT_OVERRIDE = 0;
-   virtual bool operator() (pkgCache::VerIterator const &Ver) APT_OVERRIDE { return (*this)(Ver.ParentPkg()); }
-   virtual bool operator() (pkgCache::GrpIterator const &/*Grp*/) APT_OVERRIDE { return false; }
-   virtual ~PackageMatcher();
+   bool operator() (pkgCache::PkgIterator const &Pkg) APT_OVERRIDE = 0;
+   bool operator() (pkgCache::VerIterator const &Ver) APT_OVERRIDE { return (*this)(Ver.ParentPkg()); }
+   bool operator() (pkgCache::GrpIterator const &/*Grp*/) APT_OVERRIDE { return false; }
+   ~PackageMatcher() APT_OVERRIDE;
 };
 
 // Generica like True, False, NOT, AND, OR				/*{{{*/
 class APT_PUBLIC TrueMatcher : public Matcher {
 public:
-   virtual bool operator() (pkgCache::PkgIterator const &Pkg) APT_OVERRIDE;
-   virtual bool operator() (pkgCache::GrpIterator const &Grp) APT_OVERRIDE;
-   virtual bool operator() (pkgCache::VerIterator const &Ver) APT_OVERRIDE;
+   bool operator() (pkgCache::PkgIterator const &Pkg) APT_OVERRIDE;
+   bool operator() (pkgCache::GrpIterator const &Grp) APT_OVERRIDE;
+   bool operator() (pkgCache::VerIterator const &Ver) APT_OVERRIDE;
 };
 
 class APT_PUBLIC FalseMatcher : public Matcher {
 public:
-   virtual bool operator() (pkgCache::PkgIterator const &Pkg) APT_OVERRIDE;
-   virtual bool operator() (pkgCache::GrpIterator const &Grp) APT_OVERRIDE;
-   virtual bool operator() (pkgCache::VerIterator const &Ver) APT_OVERRIDE;
+   bool operator() (pkgCache::PkgIterator const &Pkg) APT_OVERRIDE;
+   bool operator() (pkgCache::GrpIterator const &Grp) APT_OVERRIDE;
+   bool operator() (pkgCache::VerIterator const &Ver) APT_OVERRIDE;
 };
 
 class APT_PUBLIC NOTMatcher : public Matcher {
    Matcher * const matcher;
 public:
    explicit NOTMatcher(Matcher * const matcher);
-   virtual bool operator() (pkgCache::PkgIterator const &Pkg) APT_OVERRIDE;
-   virtual bool operator() (pkgCache::GrpIterator const &Grp) APT_OVERRIDE;
-   virtual bool operator() (pkgCache::VerIterator const &Ver) APT_OVERRIDE;
-   virtual ~NOTMatcher();
+   bool operator() (pkgCache::PkgIterator const &Pkg) APT_OVERRIDE;
+   bool operator() (pkgCache::GrpIterator const &Grp) APT_OVERRIDE;
+   bool operator() (pkgCache::VerIterator const &Ver) APT_OVERRIDE;
+   ~NOTMatcher() APT_OVERRIDE;
 };
 
 class APT_PUBLIC ANDMatcher : public Matcher {
@@ -72,10 +72,10 @@ public:
    ANDMatcher(Matcher * const matcher1, Matcher * const matcher2, Matcher * const matcher3, Matcher * const matcher4);
    ANDMatcher(Matcher * const matcher1, Matcher * const matcher2, Matcher * const matcher3, Matcher * const matcher4, Matcher * const matcher5);
    ANDMatcher& AND(Matcher * const matcher);
-   virtual bool operator() (pkgCache::PkgIterator const &Pkg) APT_OVERRIDE;
-   virtual bool operator() (pkgCache::GrpIterator const &Grp) APT_OVERRIDE;
-   virtual bool operator() (pkgCache::VerIterator const &Ver) APT_OVERRIDE;
-   virtual ~ANDMatcher();
+   bool operator() (pkgCache::PkgIterator const &Pkg) APT_OVERRIDE;
+   bool operator() (pkgCache::GrpIterator const &Grp) APT_OVERRIDE;
+   bool operator() (pkgCache::VerIterator const &Ver) APT_OVERRIDE;
+   ~ANDMatcher() APT_OVERRIDE;
 };
 class APT_PUBLIC ORMatcher : public Matcher {
    std::vector<Matcher *> matchers;
@@ -88,28 +88,28 @@ public:
    ORMatcher(Matcher * const matcher1, Matcher * const matcher2, Matcher * const matcher3, Matcher * const matcher4);
    ORMatcher(Matcher * const matcher1, Matcher * const matcher2, Matcher * const matcher3, Matcher * const matcher4, Matcher * const matcher5);
    ORMatcher& OR(Matcher * const matcher);
-   virtual bool operator() (pkgCache::PkgIterator const &Pkg) APT_OVERRIDE;
-   virtual bool operator() (pkgCache::GrpIterator const &Grp) APT_OVERRIDE;
-   virtual bool operator() (pkgCache::VerIterator const &Ver) APT_OVERRIDE;
-   virtual ~ORMatcher();
+   bool operator() (pkgCache::PkgIterator const &Pkg) APT_OVERRIDE;
+   bool operator() (pkgCache::GrpIterator const &Grp) APT_OVERRIDE;
+   bool operator() (pkgCache::VerIterator const &Ver) APT_OVERRIDE;
+   ~ORMatcher() APT_OVERRIDE;
 };
 									/*}}}*/
 class APT_PUBLIC PackageNameMatchesRegEx : public PackageMatcher {			/*{{{*/
 	regex_t* pattern;
 public:
 	explicit PackageNameMatchesRegEx(std::string const &Pattern);
-	virtual bool operator() (pkgCache::PkgIterator const &Pkg) APT_OVERRIDE;
-	virtual bool operator() (pkgCache::GrpIterator const &Grp) APT_OVERRIDE;
-	virtual ~PackageNameMatchesRegEx();
+	bool operator() (pkgCache::PkgIterator const &Pkg) APT_OVERRIDE;
+	bool operator() (pkgCache::GrpIterator const &Grp) APT_OVERRIDE;
+	~PackageNameMatchesRegEx() APT_OVERRIDE;
 };
 									/*}}}*/
 class APT_PUBLIC PackageNameMatchesFnmatch : public PackageMatcher {		/*{{{*/
 	const std::string Pattern;
 public:
 	explicit PackageNameMatchesFnmatch(std::string const &Pattern);
-	virtual bool operator() (pkgCache::PkgIterator const &Pkg) APT_OVERRIDE;
-	virtual bool operator() (pkgCache::GrpIterator const &Grp) APT_OVERRIDE;
-	virtual ~PackageNameMatchesFnmatch() {};
+	bool operator() (pkgCache::PkgIterator const &Pkg) APT_OVERRIDE;
+	bool operator() (pkgCache::GrpIterator const &Grp) APT_OVERRIDE;
+	~PackageNameMatchesFnmatch() APT_OVERRIDE = default;
 };
 									/*}}}*/
 class APT_PUBLIC PackageArchitectureMatchesSpecification : public PackageMatcher {	/*{{{*/
@@ -134,16 +134,18 @@ public:
 	 */
 	PackageArchitectureMatchesSpecification(std::string const &pattern, bool const isPattern = true);
 	bool operator() (char const * const &arch);
-	virtual bool operator() (pkgCache::PkgIterator const &Pkg) APT_OVERRIDE;
-	virtual ~PackageArchitectureMatchesSpecification();
+	using PackageMatcher::operator();
+	bool operator() (pkgCache::PkgIterator const &Pkg) APT_OVERRIDE;
+	~PackageArchitectureMatchesSpecification() APT_OVERRIDE;
 };
 									/*}}}*/
 class APT_PUBLIC PackageIsNewInstall : public PackageMatcher {			/*{{{*/
 	pkgCacheFile * const Cache;
 public:
 	explicit PackageIsNewInstall(pkgCacheFile * const Cache);
-	virtual bool operator() (pkgCache::PkgIterator const &Pkg) APT_OVERRIDE;
-	virtual ~PackageIsNewInstall();
+	using PackageMatcher::operator();
+	bool operator() (pkgCache::PkgIterator const &Pkg) APT_OVERRIDE;
+	~PackageIsNewInstall() APT_OVERRIDE;
 };
 									/*}}}*/
 
diff --git a/apt-pkg/contrib/mmap.cc b/apt-pkg/contrib/mmap.cc
index 642e20473..0568e1cd0 100644
--- a/apt-pkg/contrib/mmap.cc
+++ b/apt-pkg/contrib/mmap.cc
@@ -23,6 +23,7 @@
 #include <apt-pkg/macros.h>
 #include <apt-pkg/mmap.h>
 
+#include <cstdint>
 #include <cstring>
 #include <string>
 #include <errno.h>
diff --git a/apt-pkg/install-progress.cc b/apt-pkg/install-progress.cc
index 8a6d87cd2..c7f7573de 100644
--- a/apt-pkg/install-progress.cc
+++ b/apt-pkg/install-progress.cc
@@ -237,7 +237,7 @@ PackageManagerFancy::~PackageManagerFancy()
       signal(SIGWINCH, SIGWINCH_orig);
 }
 
-void PackageManagerFancy::staticSIGWINCH(int signum)
+void PackageManagerFancy::staticSIGWINCH(int /*signum*/)
 {
    SIGWINCH_flag = 1;
 }