nsSocketTransport::Open{Input,Output}Stream poorly implemented.

VERIFIED FIXED

Status

()

Core
Networking
VERIFIED FIXED
17 years ago
17 years ago

People

(Reporter: Darin Fisher, Assigned: Darin Fisher)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

17 years ago
The implementation of nsSocketTransport::OpenInputStream and OpenOutputStream
should be revised to not involve the socket transport thread.  It is completely
unnecessary.  OpenInputStream should return an nsIInputStream implementation
that when Read calls PR_Read on the socket file descriptor directly, and
likewise for OpenOutputStream.  There is no advantage to the current worker-
thread-in-conjunction-with-a-pipe scheme, so it should go away.

This is a trivial change that will significantly reduce the complexity of
nsSocketTransport.cpp.  It is also a necessary step toward providing "true"
synchronous channel i/o throughout necko.
(Assignee)

Comment 1

17 years ago
This was fixed when dougt's branch landed.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 2

17 years ago
verified
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.