Closed
Bug 377617
Opened 18 years ago
Closed 18 years ago
get rid of AtkTextChange struct
Categories
(Core :: Disability Access APIs, defect)
Core
Disability Access APIs
Tracking
()
RESOLVED
FIXED
People
(Reporter: surkov, Assigned: surkov)
References
Details
Attachments
(1 file)
(deleted),
patch
|
ginnchen+exoracle
:
review+
|
Details | Diff | Splinter Review |
Assignee | ||
Comment 1•18 years ago
|
||
Attachment #261655 -
Flags: review?(ginn.chen)
Assignee | ||
Updated•18 years ago
|
Status: NEW → ASSIGNED
Comment on attachment 261655 [details] [diff] [review]
patch
It seems we can also use
+ PRUint32 length = 1;
and don't need
+ length = 1; in else {}
for nsHyperTextAccessible::DidInsertNode
thus it will look more similar to nsHyperTextAccessible::WillDeleteNode
Attachment #261655 -
Flags: review?(ginn.chen) → review+
Assignee | ||
Comment 3•18 years ago
|
||
checked in
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
there's a typo in this patch, see
https://bugzilla.mozilla.org/show_bug.cgi?id=377756#c1
Assignee | ||
Comment 6•18 years ago
|
||
(In reply to comment #5)
> typos fixed
>
thank you, ginn
You need to log in
before you can comment on or make changes to this bug.
Description
•