Closed Bug 211808 Opened 21 years ago Closed 11 years ago

update comment

Categories

(Core :: XPCOM, defect)

x86
All
defect
Not set
trivial

Tracking

()

RESOLVED INCOMPLETE

People

(Reporter: mcsmurf, Assigned: mcsmurf)

Details

Attachments

(2 files, 1 obsolete file)

update irc comment in
xpcom/sample/nsISample.idl
Attached patch update comment (obsolete) — Splinter Review
Comment on attachment 127090 [details] [diff] [review]
update comment

Index: nsISample.idl
===================================================================
RCS file: /cvsroot/mozilla/xpcom/sample/nsISample.idl,v
retrieving revision 1.6
diff -u -r1.6 nsISample.idl
--- nsISample.idl	28 Sep 2001 20:12:59 -0000	1.6
+++ nsISample.idl	5 Jul 2003 22:06:40 -0000
@@ -50,8 +50,7 @@
  * Unix: uuidgen which comes with e2fsprogs package
  * Mac: ???
  * All platform: Using irc, connect to irc.mozilla.org and type in
- *				/join #mozilla
- *				/msg mozbot uuid
+ *				/msg botbot uuid
  *
  */
 [scriptable, uuid(7CB5B7A1-07D7-11d3-BDE2-000064657374)]
Attachment #127090 - Attachment is obsolete: true
Attachment #127091 - Flags: review?(dougt)
dought: can you also check in this then?
Comment on attachment 127091 [details] [diff] [review]
update comment #2

i think explaining what a UUID is in the context of this interface seams wrong.
 There are better places to do this.  I say remove this comment.

Thoughts?
would be ok
Comment on attachment 127091 [details] [diff] [review]
update comment #2

yeah, just remove that entire block.
Attachment #127091 - Flags: review?(dougt) → review-
Attachment #171582 - Flags: review?
Attachment #171582 - Attachment description: removed commented blcok → removed commented block
Attachment #171582 - Flags: review? → review?(dougt)
Comment on attachment 171582 [details] [diff] [review]
removed commented block

http://lxr.mozilla.org/mozilla/source/xpcom/sample/nsISample.idl#58

this is a sample of an XPCOM interface that is used by XPConnect.  Maybe we should also fix that too?
QA Contact: scc → xpcom
Attachment #171582 - Flags: review?(dougt)
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: