tree 9bb252aed1d8fec3c2bacd7e76409fdf397a6a2c
parent afd8a3631971141684685a505c1cbdd22e8c12b1
author Jean-Baptiste Lespiau <jblespiau@google.com> 1655738015 -0700
committer TensorFlower Gardener <gardener@tensorflow.org> 1655738275 -0700

Replace `tensorflow::StatusOr::ConsumeValueOrDie` with `StatusOr::value()`.

This is to align `tensorflow::StatusOr` with `absl::StatusOr`.

See https://github.com/abseil/abseil-cpp/blob/master/absl/status/statusor.h#L501-L525 for documentation. In particular:

```
  // Otherwise, if the value type supports an efficient move, it can be
  // used as follows:
  //
  //   T value = std::move(statusor).value();
  //
  // The `std::move` on statusor instead of on the whole expression enables
  // warnings about possible uses of the statusor object after the move.
```

One benefit of having the `std::move(status_or).value()` compared to `status_or.ConsumeValueOrDie()` is that the compiler will know that `status_or` should not be accessed anymore.

PiperOrigin-RevId: 456079434