diff --git a/CHANGELOG.md b/CHANGELOG.md index 6052ffea800bf..b40aeb51e06a0 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -127,9 +127,8 @@ storage::internal::ResumableUploadResource{ } ``` -That is, you need to re-order the fields **and** change increase the -`value` to reflect the number of committed bytes vs. the index in the -last committed byte. +That is, you need to re-order the fields **and** increase the `value` to reflect +the number of committed bytes vs. the index in the last committed byte. Changing the order of the fields was intentional. It will create a build failure, which is easier to detect and repair than a run-time error in diff --git a/google/cloud/storage/internal/resumable_upload_session.cc b/google/cloud/storage/internal/resumable_upload_session.cc index f38954a619bde..8a0c55344c02b 100644 --- a/google/cloud/storage/internal/resumable_upload_session.cc +++ b/google/cloud/storage/internal/resumable_upload_session.cc @@ -29,7 +29,6 @@ StatusOr ParseRangeHeader(std::string const& range) { // We expect a `Range:` header in the format described here: // https://cloud.google.com/storage/docs/json_api/v1/how-tos/resumable-upload // that is the value should match `bytes=0-[0-9]+`: - char const prefix[] = "bytes=0-"; auto constexpr kPrefixLen = sizeof(prefix) - 1; if (range.rfind(prefix, 0) != 0) {