Closed Bug 16718 Opened 25 years ago Closed 25 years ago

rename files under mozilla/mailnews/news

Categories

(MailNews Core :: Backend, defect, P3)

x86
All
defect

Tracking

(Not tracked)

VERIFIED WONTFIX

People

(Reporter: sspitzer, Assigned: sspitzer)

Details

under mozilla/mailnews/news/* we have nsNNTP*.cpp, nsNntp*.cpp and nsNews*.cpp it'd be nice to just pick one. this would be a monster change, but would be great.
carpool with leaf on this one - he can change the CVS files themselves, and coordinate the unix makefile changes
the files aren't the big pain, it is the class name cleanup. but nothing a good sed / perl script can't cure.
Actually the file name changes are the biggest part. Do to the fact that CVS is case insensitive, you won't be able to rename nsNNTP* stuff to nsNntp*. I already tried to fix this problem for news back in March and was unable to do so because of this problem. You'll need to get Leaf to do it for you and he gave me a hard time (in a very good natured way of course!) about doing it.
hey, CVS is case sensitive, it's windows that's not :)
accepting bug.
I don't see a problem with nsNews and nsNNTP both existing, for there is code in the news world that isn't just NNTP-related. (Right?) And when you're renaming, I _really_ _really_ think that we want to have nsNNTPProtocol rather than nsNntpProtocol: NNTP is an acronym, and should be capitalized as such. Don't make me come over there.
I'll do anything to prevent getting pimp slapped by shaver. nsNews and nsNNTP is good with me. this work will require leaf's help. but don't worry leaf, we probably won't get to this for a while.
Bring it on, homeslice. I'm ready for you any time.
Target Milestone: M15
Target Milestone: M15 → M20
not going to happen any time soon. moving to m20.
who are we kidding. I'm not going to get to this type of bug in 6.0 marking m30.
Target Milestone: M20 → M30
who are we kidding.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → WONTFIX
ok, will mark verified.
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.