Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "base/sync_socket.h" | 5 #include "base/sync_socket.h" |
| 6 | 6 |
| 7 #include <errno.h> | 7 #include <errno.h> |
| 8 #include <limits.h> | 8 #include <limits.h> |
| 9 #include <stdio.h> | 9 #include <stdio.h> |
| 10 #include <sys/types.h> | 10 #include <sys/types.h> |
| 11 | 11 |
| 12 #include "base/file_util.h" | 12 #include "base/file_util.h" |
| 13 #include "base/logging.h" | 13 #include "base/logging.h" |
| 14 | 14 |
| 15 | 15 |
| 16 namespace base { | 16 namespace base { |
| 17 | 17 |
| 18 const SyncSocket::Handle SyncSocket::kInvalidHandle = -1; | 18 const SyncSocket::Handle SyncSocket::kInvalidHandle = -1; |
| 19 | 19 |
| 20 SyncSocket::SyncSocket() : handle_(kInvalidHandle) { | 20 SyncSocket::SyncSocket() : handle_(kInvalidHandle) { |
| 21 NOTREACHED(); | |
| 22 } | 21 } |
| 23 | 22 |
| 24 SyncSocket::~SyncSocket() { | 23 SyncSocket::~SyncSocket() { |
| 25 } | 24 } |
| 26 | 25 |
| 27 // static | 26 // static |
| 28 bool SyncSocket::CreatePair(SyncSocket* socket_a, SyncSocket* socket_b) { | 27 bool SyncSocket::CreatePair(SyncSocket* socket_a, SyncSocket* socket_b) { |
| 29 return false; | 28 return false; |
| 30 } | 29 } |
| 31 | 30 |
| 32 bool SyncSocket::Close() { | 31 bool SyncSocket::Close() { |
| 33 return false; | 32 if (handle_ != -1) { |
|
brettw
2012/07/13 23:12:55
Can you use kInvalidHandle instead of -1 in this f
bbudge
2012/07/13 23:44:05
Done.
| |
| 33 close(handle_); | |
|
Mark Seaborn
2012/07/13 23:18:32
Please check the return value of close()
bbudge
2012/07/13 23:44:05
Done.
| |
| 34 handle_ = -1; | |
| 35 } | |
| 36 return true; | |
| 34 } | 37 } |
| 35 | 38 |
| 36 size_t SyncSocket::Send(const void* buffer, size_t length) { | 39 size_t SyncSocket::Send(const void* buffer, size_t length) { |
| 37 return 0; | 40 return -1; |
|
brettw
2012/07/13 23:12:55
Can you comment why this isn't implemented?
bbudge
2012/07/13 23:44:05
Done.
| |
| 38 } | 41 } |
| 39 | 42 |
| 40 size_t SyncSocket::Receive(void* buffer, size_t length) { | 43 size_t SyncSocket::Receive(void* buffer, size_t length) { |
| 41 return 0; | 44 return read(handle_, buffer, length); |
| 42 } | 45 } |
| 43 | 46 |
| 44 size_t SyncSocket::Peek() { | 47 size_t SyncSocket::Peek() { |
| 45 return 0; | 48 return -1; |
| 46 } | 49 } |
| 47 | 50 |
| 48 CancelableSyncSocket::CancelableSyncSocket() {} | 51 CancelableSyncSocket::CancelableSyncSocket() { |
| 52 } | |
| 53 | |
| 49 CancelableSyncSocket::CancelableSyncSocket(Handle handle) | 54 CancelableSyncSocket::CancelableSyncSocket(Handle handle) |
| 50 : SyncSocket(handle) { | 55 : SyncSocket(handle) { |
| 51 } | 56 } |
| 52 | 57 |
| 53 size_t CancelableSyncSocket::Send(const void* buffer, size_t length) { | 58 size_t CancelableSyncSocket::Send(const void* buffer, size_t length) { |
| 54 return 0; | 59 return -1; |
| 55 } | 60 } |
| 56 | 61 |
| 57 bool CancelableSyncSocket::Shutdown() { | 62 bool CancelableSyncSocket::Shutdown() { |
| 58 return false; | 63 return false; |
| 59 } | 64 } |
| 60 | 65 |
| 61 // static | 66 // static |
| 62 bool CancelableSyncSocket::CreatePair(CancelableSyncSocket* socket_a, | 67 bool CancelableSyncSocket::CreatePair(CancelableSyncSocket* socket_a, |
| 63 CancelableSyncSocket* socket_b) { | 68 CancelableSyncSocket* socket_b) { |
| 64 return SyncSocket::CreatePair(socket_a, socket_b); | 69 return SyncSocket::CreatePair(socket_a, socket_b); |
| 65 } | 70 } |
| 66 | 71 |
| 67 } // namespace base | 72 } // namespace base |
| OLD | NEW |