Closed
Bug 246337
Opened 20 years ago
Closed 20 years ago
nsIProxy should be cvs removed
Categories
(Core :: Networking, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.8alpha2
People
(Reporter: Biesinger, Assigned: darin.moz)
References
()
Details
Attachments
(1 file)
5.99 KB,
patch
|
Biesinger
:
review+
|
Details | Diff | Splinter Review |
39 The nsIProxy interface allows setting and getting of proxy host and port. 49 [scriptable, uuid(0492D011-CD2F-11d2-B013-006097BFC036)] 50 interface nsIProxy : nsISupports 51 { 52 53 };
Assignee | ||
Comment 1•20 years ago
|
||
nsIProxy is unused. It should be removed from the tree. It was replaced with nsIProxyInfo.
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.8alpha2
Assignee | ||
Comment 2•20 years ago
|
||
this patch removes all references to nsIProxy in the source. it was effectively unused, but this patch is needed so that cvs removing it does not break the build.
Assignee | ||
Updated•20 years ago
|
Attachment #150526 -
Flags: review?(cbiesinger)
Reporter | ||
Comment 3•20 years ago
|
||
Comment on attachment 150526 [details] [diff] [review] v1 patch ah - nsISOCKSSocketInfo referencing it was what confused me. +++ socket/base/nsISOCKS4SocketInfo.idl 11 Jun 2004 14:55:15 -0000 this entire file is unused (http://lxr.mozilla.org/seamonkey/search?string=socks4socketinfo); you may want to remove it here... otherwise, bug 134105's patch contains removal of it.
Attachment #150526 -
Flags: review?(cbiesinger) → review+
Assignee | ||
Comment 4•20 years ago
|
||
Ah.. I didn't even check to see if it was used. I'll just remove it here then.
Assignee | ||
Comment 5•20 years ago
|
||
fixed-on-trunk
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Summary: nsIProxy documentation does not match reality → nsIProxy should be cvs removed
You need to log in
before you can comment on or make changes to this bug.
Description
•