diff options
author | Xiao Ma <xiaom@google.com> | 2019-10-17 15:54:06 +0900 |
---|---|---|
committer | Xiao Ma <xiaom@google.com> | 2019-10-17 15:54:06 +0900 |
commit | df4c156e6ef04454cc99087e75f570625d37ed70 (patch) | |
tree | 1f88dd4ef89dfdb862e7385b625b34a3eb81c097 /src/android/net/util/PacketReader.java | |
parent | 3c9f687c97393a15d238a3272f62c595052e6b2e (diff) |
Optimize FdEventsReader to make it safer to use.
Update the start() and createAndRegisterFd() to boolean type, to make
sure we won't return null socket when failing to create socket.
Throwing IllegalStateException if any caller is calling the FdEventsReader#
start/stop from the different thread, to make it immediately obvious.
Bug: None
Test: atest NetworkStackTests
Change-Id: I398b24da34b8fc9da7e8a9d07508c4d731c4a437
Diffstat (limited to 'src/android/net/util/PacketReader.java')
-rw-r--r-- | src/android/net/util/PacketReader.java | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/android/net/util/PacketReader.java b/src/android/net/util/PacketReader.java index 4aec6b6..0be7187 100644 --- a/src/android/net/util/PacketReader.java +++ b/src/android/net/util/PacketReader.java @@ -28,8 +28,6 @@ import java.io.FileDescriptor; * * TODO: rename this class to something more correctly descriptive (something * like [or less horrible than] FdReadEventsHandler?). - * - * @hide */ public abstract class PacketReader extends FdEventsReader<byte[]> { |