Closed Bug 137302 Opened 24 years ago Closed 24 years ago

Fix some typos in nsIFrame documentation

Categories

(Core :: Layout, defect)

x86
All
defect
Not set
trivial

Tracking

()

VERIFIED FIXED

People

(Reporter: hwaara, Assigned: hwaara)

Details

Attachments

(1 file)

While reading nsIFrame.h I noticed and corrected some minor typos and such. Patch forthcoming.
Attached patch PatchSplinter Review
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+
Patch checked in. Out of curiousity, is brendan's rs= always implied when making trivial comment-only changes?
Status: NEW → RESOLVED
Closed: 24 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
Changing QA contact
QA Contact: petersen → moied
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.

Attachment

General

Creator:
Created:
Updated:
Size: