diff options
| author | George Burgess IV <gbiv@google.com> | 2020-04-01 13:34:21 -0700 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2020-04-04 10:21:34 +0000 |
| commit | c48581a0b26c77f900aa7b82bd13f4c6691d474f (patch) | |
| tree | 33419b12b51153a9aaae4270b0705976d9240726 /test_http_server.cc | |
| parent | 9d5a61d7c71beb655e67eaf5c685e7add8f7768d (diff) | |
update_engine: remove an unreachable return
We're trying to turn -Wunreachable-code-return on by default, and this
unreachable `return` statement was flagged. The code above this is
pretty clearly an infloop, so this return statement doesn't seem super
useful. If someone adds a `break` or similar later, the compiler will
complain (in -Werror form) about a missing `return` statement.
BUG=chromium:1066998
TEST=CQ
Change-Id: I9c87e256fa0c850bcb8cec17cbe694fc6bb0ef66
Reviewed-on: https://chromium-review.googlesource.com/c/aosp/platform/system/update_engine/+/2133017
Reviewed-by: Sen Jiang <senj@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
Commit-Queue: George Burgess <gbiv@chromium.org>
Diffstat (limited to 'test_http_server.cc')
| -rw-r--r-- | test_http_server.cc | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/test_http_server.cc b/test_http_server.cc index 4536f372..4fc89e53 100644 --- a/test_http_server.cc +++ b/test_http_server.cc @@ -658,5 +658,4 @@ int main(int argc, char** argv) { LOG(FATAL) << "ERROR on accept"; HandleConnection(client_fd); } - return 0; } |