Closed Bug 99156 Opened 19 years ago Closed 19 years ago

Freeze nsIProgrammingLanguage

Categories

(Core :: XPCOM, defect)

x86
Windows 2000
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla1.0

People

(Reporter: dougt, Assigned: dougt)

References

Details

Attachments

(1 file)

yes, the nsIClassInfo fan-out requires this interface.  Do we have to freeze if
this interface only grows with enum constants?
Blocks: 98278
I say we can freeze this in the sense that no methods or attributes should ever 
be added. We'd want to stop anyone from doing that! I think we should understand 
that new consts are allowed even though this is considered frozen. Adding 
constants will not break anything. Adjust the rules to the reality.
jband, can you review this patch which freezes this interface?
hold off on that review.  Let me add some comments so that people know what the
hell this interface does. :-)
Index: nsIProgrammingLanguage.idl
===================================================================
RCS file: /cvsroot/mozilla/xpcom/base/nsIProgrammingLanguage.idl,v
retrieving revision 1.3
diff -u -r1.3 nsIProgrammingLanguage.idl
--- nsIProgrammingLanguage.idl	2001/05/09 20:51:53	1.3
+++ nsIProgrammingLanguage.idl	2001/09/13 20:45:19
@@ -41,6 +41,11 @@
 #endif
 %}
 
+/**
+ * Enumeration of Programming Languages
+ * @status FROZEN
+ */
+
 [scriptable, uuid(ea604e90-40ba-11d5-90bb-0010a4e73d9a)]
 interface nsIProgrammingLanguage : nsISupports
 {

can I get a r=, jband?
r/rs=jband
Comment on attachment 49060 [details] [diff] [review]
Addes @status line to interface

r=rpotts

looks good to me.
Attachment #49060 - Flags: review+
Target Milestone: --- → mozilla1.0
I checked this in already.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.