Skip to content

Commit c2be021

Browse files
committed
clang-tidy: use C++ casts
Signed-off-by: Rosen Penev <[email protected]>
1 parent f787a4c commit c2be021

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

app/exiv2.cpp

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -742,12 +742,12 @@ int Params::evalDelete(const std::string& optArg) {
742742
switch (action_) {
743743
case Action::none:
744744
action_ = Action::erase;
745-
target_ = CommonTarget(0);
745+
target_ = static_cast<CommonTarget>(0);
746746
// fallthrough
747747
case Action::erase: {
748748
const auto rc = parseCommonTargets(optArg, "erase");
749749
if (rc > 0) {
750-
target_ |= CommonTarget(rc);
750+
target_ |= static_cast<CommonTarget>(rc);
751751
return 0;
752752
} else {
753753
return 1;
@@ -764,12 +764,12 @@ int Params::evalExtract(const std::string& optArg) {
764764
case Action::none:
765765
case Action::modify:
766766
action_ = Action::extract;
767-
target_ = CommonTarget(0);
767+
target_ = static_cast<CommonTarget>(0);
768768
// fallthrough
769769
case Action::extract: {
770770
const auto rc = parseCommonTargets(optArg, "extract");
771771
if (rc > 0) {
772-
target_ |= CommonTarget(rc);
772+
target_ |= static_cast<CommonTarget>(rc);
773773
return 0;
774774
} else {
775775
return 1;
@@ -786,12 +786,12 @@ int Params::evalInsert(const std::string& optArg) {
786786
case Action::none:
787787
case Action::modify:
788788
action_ = Action::insert;
789-
target_ = CommonTarget(0);
789+
target_ = static_cast<CommonTarget>(0);
790790
// fallthrough
791791
case Action::insert: {
792792
const auto rc = parseCommonTargets(optArg, "insert");
793793
if (rc > 0) {
794-
target_ |= CommonTarget(rc);
794+
target_ |= static_cast<CommonTarget>(rc);
795795
return 0;
796796
} else {
797797
return 1;
@@ -1139,7 +1139,7 @@ void printUnrecognizedArgument(const char argc, const std::string& action) {
11391139

11401140
int64_t parseCommonTargets(const std::string& optArg, const std::string& action) {
11411141
int64_t rc = 0;
1142-
Params::CommonTarget target = Params::CommonTarget(0);
1142+
Params::CommonTarget target = static_cast<Params::CommonTarget>(0);
11431143
Params::CommonTarget all = Params::ctExif | Params::ctIptc | Params::ctComment | Params::ctXmp;
11441144
Params::CommonTarget extra = Params::ctXmpSidecar | Params::ctExif | Params::ctIptc | Params::ctXmp;
11451145
for (size_t i = 0; rc == 0 && i < optArg.size(); ++i) {
@@ -1175,7 +1175,7 @@ int64_t parseCommonTargets(const std::string& optArg, const std::string& action)
11751175
target |= extra; // -eX
11761176
if (i > 0) { // -eXX or -iXX
11771177
target |= Params::ctXmpRaw;
1178-
target = Params::CommonTarget(target & ~extra); // turn off those bits
1178+
target = static_cast<Params::CommonTarget>(target & ~extra); // turn off those bits
11791179
}
11801180
break;
11811181

@@ -1196,7 +1196,7 @@ int64_t parseCommonTargets(const std::string& optArg, const std::string& action)
11961196
break;
11971197
}
11981198
}
1199-
return rc ? rc : int64_t(target);
1199+
return rc ? rc : static_cast<int64_t>(target);
12001200
}
12011201

12021202
int parsePreviewNumbers(Params::PreviewNumbers& previewNumbers, const std::string& optArg, int j) {

0 commit comments

Comments
 (0)