299932 - fix nsIMimeEmitter to use proper case

Status: RESOLVED FIXED

(MailNews Core :: MIME, defect)

Description

[timeless]

this interface is only used internally so now is the time to fix it.

Comment 1

[Magnus Melin [:mkmelin]]

Attachment: proposed fix

Change to lowerCamelCase and clean up whitespace/comments. 
Also the same for nsIMsgQuote.

Comment 2

[David :Bienvenu]

Comment on attachment 342616 [details] [diff] [review]
proposed fix

thanks for doing this.

can't this be an attribute?

+    void setOutputListener(in nsIStreamListener listener);
 
     //Get the output listener to allow access to it from mime
-    nsIStreamListener    GetOutputListener();
+    nsIStreamListener getOutputListener();

have you checked there are no js callers that would be affected by the case change?

Comment 3

[Magnus Melin [:mkmelin]]

Attachment: proposed fix, v2

Yes, looks like it can be an attribute. 
AFAICT, there are no js callers at all for this interface.

Comment 4

[David :Bienvenu]

Comment on attachment 342622 [details] [diff] [review]
proposed fix, v2

ok, thx, Magnus.

Comment 5

[Magnus Melin [:mkmelin]]

changeset:   579:f6239bbf466f
http://hg.mozilla.org/comm-central/rev/f6239bbf466f

->FIXED