summaryrefslogtreecommitdiff
path: root/src/buildtool/build_engine/expression/expression.cpp
diff options
context:
space:
mode:
authorMaksim Denisov <denisov.maksim@huawei.com>2024-09-30 16:16:20 +0200
committerMaksim Denisov <denisov.maksim@huawei.com>2024-10-07 13:37:39 +0200
commit030af3e7af210617df00bf850476df6b46fb004d (patch)
tree6bfaff768b035c382fe0db07909ecd1f908f1eaf /src/buildtool/build_engine/expression/expression.cpp
parentce23db59c6399199fa55b4b7dc8880522e2f1bca (diff)
downloadjustbuild-030af3e7af210617df00bf850476df6b46fb004d.tar.gz
Enable readability-* checks.
Diffstat (limited to 'src/buildtool/build_engine/expression/expression.cpp')
-rw-r--r--src/buildtool/build_engine/expression/expression.cpp18
1 files changed, 13 insertions, 5 deletions
diff --git a/src/buildtool/build_engine/expression/expression.cpp b/src/buildtool/build_engine/expression/expression.cpp
index 7d0f55b5..cbb6a18d 100644
--- a/src/buildtool/build_engine/expression/expression.cpp
+++ b/src/buildtool/build_engine/expression/expression.cpp
@@ -246,11 +246,19 @@ auto Expression::ComputeHash() const noexcept -> std::string {
IsResult() or IsNode() or IsName()) {
// just hash the JSON representation, but prepend "@" for artifact,
// "=" for result, "#" for node, and "$" for name.
- std::string prefix{IsArtifact() ? "@"
- : IsResult() ? "="
- : IsNode() ? "#"
- : IsName() ? "$"
- : ""};
+ std::string prefix;
+ if (IsArtifact()) {
+ prefix = "@";
+ }
+ else if (IsResult()) {
+ prefix = "=";
+ }
+ else if (IsNode()) {
+ prefix = "#";
+ }
+ else if (IsName()) {
+ prefix = "$";
+ }
hash = hash_function.PlainHashData(prefix + ToString()).Bytes();
}
else {