diff options
| author | thakis@chromium.org <thakis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-23 00:37:46 +0000 |
|---|---|---|
| committer | thakis@chromium.org <thakis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-23 00:37:46 +0000 |
| commit | 70eb657d0e9575764e6d744a2fd36f7f7249c799 (patch) | |
| tree | cbe69d505bf6978e1b0a70c9deb13e33c677b659 /base/message_loop_unittest.cc | |
| parent | 44a34186a63080c37c70bddc0ecd2c5efc9d7fab (diff) | |
| download | chromium_src-70eb657d0e9575764e6d744a2fd36f7f7249c799.zip chromium_src-70eb657d0e9575764e6d744a2fd36f7f7249c799.tar.gz chromium_src-70eb657d0e9575764e6d744a2fd36f7f7249c799.tar.bz2 | |
Clang: Do not ignore result of HANDLE_EINTR.
Review URL: http://codereview.chromium.org/2805026
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@50556 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/message_loop_unittest.cc')
| -rw-r--r-- | base/message_loop_unittest.cc | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/base/message_loop_unittest.cc b/base/message_loop_unittest.cc index 6208e99..da0013c 100644 --- a/base/message_loop_unittest.cc +++ b/base/message_loop_unittest.cc @@ -1578,8 +1578,10 @@ TEST(MessageLoopTest, FileDescriptorWatcherOutlivesMessageLoop) { // and don't run the message loop, just destroy it. } } - HANDLE_EINTR(close(pipefds[0])); - HANDLE_EINTR(close(pipefds[1])); + if (HANDLE_EINTR(close(pipefds[0])) < 0) + PLOG(ERROR) << "close"; + if (HANDLE_EINTR(close(pipefds[1])) < 0) + PLOG(ERROR) << "close"; } TEST(MessageLoopTest, FileDescriptorWatcherDoubleStop) { @@ -1601,8 +1603,10 @@ TEST(MessageLoopTest, FileDescriptorWatcherDoubleStop) { controller.StopWatchingFileDescriptor(); } } - HANDLE_EINTR(close(pipefds[0])); - HANDLE_EINTR(close(pipefds[1])); + if (HANDLE_EINTR(close(pipefds[0])) < 0) + PLOG(ERROR) << "close"; + if (HANDLE_EINTR(close(pipefds[1])) < 0) + PLOG(ERROR) << "close"; } } // namespace |