Closed Bug 99592 Opened 18 years ago Closed 13 years ago

hacking the DOM feedback

Categories

(Developer Documentation :: General, defect, P4)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: axel, Assigned: fabian)

References

()

Details

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.