minor clang-tidy fixes (#2213)

neheb · web-flow · commit d466c1e9eb7d · 2022-04-16T11:29:15.000+02:00
* clang-tidy: add const to member function

Signed-off-by: Rosen Penev &lt;rosenp@gmail.com&gt;

* clang-tidy: remove duplicate access specifier

Signed-off-by: Rosen Penev &lt;rosenp@gmail.com&gt;
diff --git 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
@@ -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
@@ -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.
    */

Original file line number	Diff line number	Diff line change
`@@ -73,7 +73,7 @@ class Task {`
`73`	`73`	`return bResult;`
`74`	`74`	`}`
`75`	`75`
`76`		`- bool binary() {`
	`76`	`+ bool binary() const {`
`77`	`77`	`return binary_;`
`78`	`78`	`}`
`79`	`79`