Closed
Bug 137302
Opened 24 years ago
Closed 24 years ago
Fix some typos in nsIFrame documentation
Categories
(Core :: Layout, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: hwaara, Assigned: hwaara)
Details
Attachments
(1 file)
|
3.35 KB,
patch
|
bzbarsky
:
review+
|
Details | Diff | Splinter Review |
While reading nsIFrame.h I noticed and corrected some minor typos and such.
Patch forthcoming.
| Assignee | ||
Comment 1•24 years ago
|
||
| Assignee | ||
Comment 2•24 years ago
|
||
I assume I don't need sr= for a comment-only change. Now looking for reviewers...
Comment 3•24 years ago
|
||
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•24 years ago
|
||
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
Comment 5•24 years ago
|
||
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
You need to log in
before you can comment on or make changes to this bug.
Description
•