Closed Bug 99592 Opened 18 years ago Closed 13 years ago
hacking the DOM feedback
Here goes some feedback to the DOM hacking page. First of all, link this bug ;-)Chapter 1, (I tend to call it nsINutshell), should at least be a separate doc. If we don't have such a document for XPCOM already. 1.B misses reference counting alltogether. There is discussion of lifetime and interface pointers, probably that's good, but link to the XPCOM ownership guide. 1.C: nsCOMPtrs are an extension of C++ auto_ptr, to deal with decreasing the reference count instead of just deleating the object. 1.D: > The first choice is to use a "Getter", the second choice is to use a static > cast on the "this" pointer. This isn't really a choice. Those are two completely different problems, once again, probably extensivly handled in the ownership guide. (That getters addref by definition, for example). 1.E.b: say something about the uuid. I'm not sure if it's good to have the actual one you got for nsIFabian, but either way, you should say what you're doing. 1.E.c: Makefiles have mandatory tab use in just one place, that's one tab for commands for a rule. All other whitespace is white. How about a typical line of REQUIRES for DOM code? 'nuff said for this time.
not checked in but here are the changes I did - split in two: xpcom intro vs classinfo. Further doc will be added after the classinfo doc - fixed some issues Pike raised, specifically 1.B, 1.C, both 1.E. Still need to add the links. 1.D will take longer. Also addressing dbradley's comments. I hope to be done by friday.
Status: NEW → ASSIGNED
Priority: -- → P2
Checked in. The XPCOM intro is now at http://mozilla.org/docs/dom/mozilla/xpcomintro.html and the class info doc at http://mozilla.org/docs/dom/mozilla/hacking.html
Triaging my bugs. The main problems have been fixed, leaving this bug open anyway, but marking P4 and future. If there are more issues raised I'll fix them as they come.
Priority: P2 → P4
Target Milestone: --- → Future
accepting QA for mozilla developer docs. some of these bugs have been around for a _long_ time. Reporters, would you please review the bugs, see if the issues have been resolved, and close bugs appropriately. I will do a full review of all bugs not touched in one week (8th April). Thanks. </spam>
QA Contact: endico → imajes
Tweaking URL to the XPCOM doc (I doubt anyone's going to report bugs for the DOMClassInfo doc ;-). I'd like to keep this bug open. Thanks imajes for volunteering, this is great!
Well, since no new issues were raised over the last five years, I guess you nailed it perfectly.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Component: Mozilla Developer → Documentation Requests
Product: Documentation → Mozilla Developer Center
Target Milestone: Future → ---
Component: Documentation Requests → Documentation
Component: Documentation → General
Product: Mozilla Developer Network → Developer Documentation
You need to log in before you can comment on or make changes to this bug.