From d466c1e9eb7dea9b7da540ad92c2f22efed24eb5 Mon Sep 17 00:00:00 2001
From: Rosen Penev <rosenp@gmail.com>
Date: Sat, 16 Apr 2022 02:29:15 -0700
Subject: [PATCH] minor clang-tidy fixes (#2213)

* clang-tidy: add const to member function

Signed-off-by: Rosen Penev <rosenp@gmail.com>

* clang-tidy: remove duplicate access specifier

Signed-off-by: Rosen Penev <rosenp@gmail.com>
---
 app/actions.hpp      | 2 +-
 app/exiv2app.hpp     | 1 -
 samples/metacopy.hpp | 1 -
 3 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/app/actions.hpp b/app/actions.hpp
index 809b8150..10751e7c 100644
--- a/app/actions.hpp
+++ b/app/actions.hpp
@@ -73,7 +73,7 @@ class Task {
     return bResult;
   }
 
-  bool binary() {
+  bool binary() const {
     return binary_;
   }
 
diff --git a/app/exiv2app.hpp b/app/exiv2app.hpp
index f93db850..c12fce61 100644
--- a/app/exiv2app.hpp
+++ b/app/exiv2app.hpp
@@ -217,7 +217,6 @@ class Params : public Util::Getopt {
 
   Params();
 
- private:
   //! @name Helpers
   //@{
   int setLogLevel(const std::string& optarg);
diff --git a/samples/metacopy.hpp b/samples/metacopy.hpp
index aafa22e0..003ea66c 100644
--- a/samples/metacopy.hpp
+++ b/samples/metacopy.hpp
@@ -22,7 +22,6 @@ class Params : public Util::Getopt {
   std::string read_;      //!< Source file
   std::string write_;     //!< Destination file
 
- public:
   /*!
     @brief Default constructor. Note that optstring_ is initialized here.
    */