diff options
| author | Maksim Denisov <denisov.maksim@huawei.com> | 2024-07-09 15:28:37 +0200 |
|---|---|---|
| committer | Maksim Denisov <denisov.maksim@huawei.com> | 2024-07-12 15:13:52 +0200 |
| commit | f0ac970bac223f64e287d9b28f75b0d8da7de706 (patch) | |
| tree | c0a0675d862bc382519ff41d454fb4365dc53f54 /src/buildtool/execution_api/local/local_action.hpp | |
| parent | 5057bd9e9a55fd8423ad7112086a3cafd3162e2c (diff) | |
| download | justbuild-f0ac970bac223f64e287d9b28f75b0d8da7de706.tar.gz | |
Return std::nullopt if creation of an action digest fails
...instead of dereferencing nullptr.
Diffstat (limited to 'src/buildtool/execution_api/local/local_action.hpp')
| -rw-r--r-- | src/buildtool/execution_api/local/local_action.hpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/buildtool/execution_api/local/local_action.hpp b/src/buildtool/execution_api/local/local_action.hpp index 7999da33..582f9b8e 100644 --- a/src/buildtool/execution_api/local/local_action.hpp +++ b/src/buildtool/execution_api/local/local_action.hpp @@ -18,6 +18,7 @@ #include <chrono> #include <map> #include <memory> +#include <optional> #include <string> #include <unordered_map> #include <utility> // std::move @@ -97,7 +98,7 @@ class LocalAction final : public IExecutionAction { [[nodiscard]] auto CreateActionDigest(bazel_re::Digest const& exec_dir, bool do_not_cache) - -> bazel_re::Digest { + -> std::optional<bazel_re::Digest> { auto const env_vars = BazelMsgFactory::CreateMessageVectorFromMap< bazel_re::Command_EnvironmentVariable>(env_vars_); |