Closed Bug 377617 Opened 17 years ago Closed 17 years ago

get rid of AtkTextChange struct

Categories

(Core :: Disability Access APIs, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: surkov, Assigned: surkov)

References

Details

Attachments

(1 file)

 
Attached patch patchSplinter Review
Attachment #261655 - Flags: review?(ginn.chen)
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+
checked in
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
there's a typo in this patch, see 
https://bugzilla.mozilla.org/show_bug.cgi?id=377756#c1
typos fixed
(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.

Attachment

General

Created:
Updated:
Size: