clazy: trivial types to value

Signed-off-by: Rosen Penev <[email protected]>

Author: neheb

src/convert.cpp

@@ -62,7 +62,7 @@ bool convertStringCharsetWindows(std::string& str, std::string_view from, std::s
   exactly one entry, \em value is set to this entry, without the qualifier.
   The return code indicates if the operation was successful.
  */
-bool getTextValue(std::string& value, const Exiv2::XmpData::iterator& pos);
+bool getTextValue(std::string& value, Exiv2::XmpData::iterator pos);
 }  // namespace
 
 // *****************************************************************************
@@ -1589,7 +1589,7 @@ bool convertStringCharsetWindows(std::string& str, std::string_view from, std::s
 }
 
 #endif  // EXV_HAVE_ICONV
-bool getTextValue(std::string& value, const XmpData::iterator& pos) {
+bool getTextValue(std::string& value, XmpData::iterator pos) {
   if (pos->typeId() == langAlt) {
     // get the default language entry without x-default qualifier
     value = pos->toString(0);

src/makernote_int.cpp

@@ -41,7 +41,7 @@ namespace fs = std::filesystem;
 namespace {
 // Todo: Can be generalized further - get any tag as a string/long/...
 //! Get the Value for a tag within a particular group
-const Exiv2::Value* getExifValue(Exiv2::Internal::TiffComponent* pRoot, const uint16_t& tag, const Exiv2::IfdId& group);
+const Exiv2::Value* getExifValue(Exiv2::Internal::TiffComponent* pRoot, uint16_t tag, Exiv2::IfdId group);
 //! Get the model name from tag Exif.Image.Model
 std::string getExifModel(Exiv2::Internal::TiffComponent* pRoot);
 
@@ -1036,8 +1036,7 @@ int sonyMisc3cSelector(uint16_t /*tag*/, const byte* /*pData*/, size_t /*size*/,
 // *****************************************************************************
 // local definitions
 namespace {
-const Exiv2::Value* getExifValue(Exiv2::Internal::TiffComponent* pRoot, const uint16_t& tag,
-                                 const Exiv2::IfdId& group) {
+const Exiv2::Value* getExifValue(Exiv2::Internal::TiffComponent* pRoot, uint16_t tag, Exiv2::IfdId group) {
   Exiv2::Internal::TiffFinder finder(tag, group);
   if (!pRoot)
     return nullptr;

src/tiffimage_int.hpp

@@ -139,7 +139,7 @@ class TiffHeader : public TiffHeaderBase {
 using TiffGroupKey = std::pair<uint32_t, IfdId>;
 
 struct TiffGroupKey_hash {
-  std::size_t operator()(const TiffGroupKey& pair) const noexcept {
+  std::size_t operator()(TiffGroupKey pair) const noexcept {
     return std::hash<uint64_t>{}(static_cast<uint64_t>(pair.first) << 32 | static_cast<uint64_t>(pair.second));
   }
 };

src/xmp.cpp

@@ -206,16 +206,16 @@ class FindXmpdatum {
 
 #ifdef EXV_HAVE_XMP_TOOLKIT
 //! Convert XMP Toolkit struct option bit to Value::XmpStruct
-Exiv2::XmpValue::XmpStruct xmpStruct(const XMP_OptionBits& opt);
+Exiv2::XmpValue::XmpStruct xmpStruct(XMP_OptionBits opt);
 
 //! Convert Value::XmpStruct to XMP Toolkit array option bits
 XMP_OptionBits xmpArrayOptionBits(Exiv2::XmpValue::XmpStruct xs);
 
 //! Convert XMP Toolkit array option bits to array TypeId
-Exiv2::TypeId arrayValueTypeId(const XMP_OptionBits& opt);
+Exiv2::TypeId arrayValueTypeId(XMP_OptionBits opt);
 
 //! Convert XMP Toolkit array option bits to Value::XmpArrayType
-Exiv2::XmpValue::XmpArrayType xmpArrayType(const XMP_OptionBits& opt);
+Exiv2::XmpValue::XmpArrayType xmpArrayType(XMP_OptionBits opt);
 
 //! Convert Value::XmpArrayType to XMP Toolkit array option bits
 XMP_OptionBits xmpArrayOptionBits(Exiv2::XmpValue::XmpArrayType xat);
@@ -225,7 +225,7 @@ XMP_OptionBits xmpFormatOptionBits(Exiv2::XmpParser::XmpFormatFlags flags);
 
 //! Print information about a parsed XMP node
 void printNode(const std::string& schemaNs, const std::string& propPath, const std::string& propValue,
-               const XMP_OptionBits& opt);
+               XMP_OptionBits opt);
 
 //! Make an XMP key from a schema namespace and property path
 Exiv2::XmpKey::UniquePtr makeXmpKey(const std::string& schemaNs, const std::string& propPath);
@@ -904,7 +904,7 @@ int XmpParser::encode(std::string& /*xmpPacket*/, const XmpData& xmpData, uint16
 // local definitions
 namespace {
 #ifdef EXV_HAVE_XMP_TOOLKIT
-Exiv2::XmpValue::XmpStruct xmpStruct(const XMP_OptionBits& opt) {
+Exiv2::XmpValue::XmpStruct xmpStruct(XMP_OptionBits opt) {
   Exiv2::XmpValue::XmpStruct var(Exiv2::XmpValue::xsNone);
   if (XMP_PropIsStruct(opt)) {
     var = Exiv2::XmpValue::xsStruct;
@@ -924,7 +924,7 @@ XMP_OptionBits xmpArrayOptionBits(Exiv2::XmpValue::XmpStruct xs) {
   return var;
 }
 
-Exiv2::TypeId arrayValueTypeId(const XMP_OptionBits& opt) {
+Exiv2::TypeId arrayValueTypeId(XMP_OptionBits opt) {
   Exiv2::TypeId typeId(Exiv2::invalidTypeId);
   if (XMP_PropIsArray(opt)) {
     if (XMP_ArrayIsAlternate(opt))
@@ -937,7 +937,7 @@ Exiv2::TypeId arrayValueTypeId(const XMP_OptionBits& opt) {
   return typeId;
 }
 
-Exiv2::XmpValue::XmpArrayType xmpArrayType(const XMP_OptionBits& opt) {
+Exiv2::XmpValue::XmpArrayType xmpArrayType(XMP_OptionBits opt) {
   return Exiv2::XmpValue::xmpArrayType(arrayValueTypeId(opt));
 }
 
@@ -986,7 +986,7 @@ XMP_OptionBits xmpFormatOptionBits(Exiv2::XmpParser::XmpFormatFlags flags) {
 
 #ifdef EXIV2_DEBUG_MESSAGES
 void printNode(const std::string& schemaNs, const std::string& propPath, const std::string& propValue,
-               const XMP_OptionBits& opt) {
+               XMP_OptionBits opt) {
   static bool first = true;
   if (first) {
     first = false;
@@ -1041,7 +1041,7 @@ void printNode(const std::string& schemaNs, const std::string& propPath, const s
   std::cout << '\n';
 }
 #else
-void printNode(const std::string&, const std::string&, const std::string&, const XMP_OptionBits&) {
+void printNode(const std::string&, const std::string&, const std::string&, XMP_OptionBits) {
 }
 #endif  // EXIV2_DEBUG_MESSAGES