Fix some typos in nsIFrame documentation

VERIFIED FIXED

Status

()

Core
Layout
--
trivial
VERIFIED FIXED
16 years ago
16 years ago

People

(Reporter: Håkan Waara, Assigned: Håkan Waara)

Tracking

Trunk
x86
All
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

16 years ago
While reading nsIFrame.h I noticed and corrected some minor typos and such.
Patch forthcoming.
(Assignee)

Comment 1

16 years ago
Created attachment 79074 [details] [diff] [review]
Patch
(Assignee)

Comment 2

16 years ago
I assume I don't need sr= for a comment-only change.  Now looking for reviewers...
Comment on attachment 79074 [details] [diff] [review]
Patch

r=bzbarsky.  I believe Brendan has an rs=brendan for spelling fixes to comments
policy...
Attachment #79074 - Flags: review+
(Assignee)

Comment 4

16 years ago
Patch checked in.  Out of curiousity, is brendan's rs= always implied when
making trivial comment-only changes?
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
I don't give blanket rs= to anyone, esp. someone I've never rs'd before. 
Whoever said otherwise wasn't quoting me accurately.  Since you cc'd me, I took
a  look after the fact and found a missing period at the end of a sentence that
you could have fixed, and lots of 80th column violations (pre-existing too, but
worth fixing as you go?).  My 2 cents, ping me again once when you want an rs=.

/be

Comment 6

16 years ago
Changing QA contact
QA Contact: petersen → moied

Comment 7

16 years ago
verified fix checked in CVS (rev 3.203)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.