Skip to content

Commit e2ee133

Browse files
committed
pass bool by value
No sense in passing by reference Signed-off-by: Rosen Penev <[email protected]>
1 parent c74ae5a commit e2ee133

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

include/exiv2/xmp_exiv2.hpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ class EXIV2API Xmpdatum : public Metadatum {
6262
@brief Assign a boolean \em value to the %Xmpdatum.
6363
Translates the value to a string "true" or "false".
6464
*/
65-
Xmpdatum& operator=(const bool& value);
65+
Xmpdatum& operator=(bool value);
6666
/*!
6767
@brief Assign a \em value of any type with an output operator
6868
to the %Xmpdatum. Calls operator=(const std::string&).
@@ -399,7 +399,7 @@ class EXIV2API XmpParser {
399399
// *****************************************************************************
400400
// free functions, template and inline definitions
401401

402-
inline Xmpdatum& Xmpdatum::operator=(const bool& value) {
402+
inline Xmpdatum& Xmpdatum::operator=(bool value) {
403403
return operator=(value ? "True" : "False");
404404
}
405405

0 commit comments

Comments
 (0)