72310 - Gopher - turn into module

Status: VERIFIED INVALID

(Core :: Networking, defect)

Description

[[e-mail]]

Comments in bug 49334 say that gopher is not a module, but it could be.

I think the case is strong for this, here are my reasons:

1- mozilla should have a narrow, focused core of supported protocols. this make 
testing easier (do we even have a gopher test suite right now?)

2- footprint considerations for embedding and overall footprint apply here. 

3- modularity is good, users would be able to pick just the modules they 
want/need.

Comment 1

[Bradley Baetz]

It is in a module - I think I discussed this on IRC rather than in the bug. Its
in nsNetModule2, which contains gopher, finger, datetime and ftp only. It may be
possible to move the gopher streamconverter into that module from the main one,
and maybe put ftp back into the main module.

Comment 2

[Bradley Baetz]

[e-mail]: This is already done, unless I misunderstood you.
Please reopen this bug if this is the case.

Comment 3

[Bradley Baetz]

[e-mail]: This is already done, unless I misunderstood you.
Please reopen this bug if this is the case.

Comment 4

[benc]

VERIFIED:
I'll reopen if this becomes an issue.