Closed Bug 782622 Opened 12 years ago Closed 12 years ago

Don't try returning nsresult in place of bool (accessible/)

Categories

(Core :: Disability Access APIs, defect)

Product:
Core
Component:
Disability Access APIs
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

()

Status:
RESOLVED DUPLICATE of bug 776481

People

(Reporter: ayg, Assigned: ayg)

References

Details

Attachments

(1 file)

Patch
1.62 KB, patch
Details | Diff | Splinter Review 
      No description provided.
Attached patch Patch — — Splinter Review 
It's not obvious to me what we should really return here, so I preserved existing behavior (with comments).  NS_ERROR_FAILURE is nonzero, therefore boolean true, and NS_OK is boolean false.
Attachment #651735 - Flags: review?(surkov.alexander)
this one is a dupe of bug 776481. Please finish the patch addressing comments in bug 782622 (since it seems Mato is not active on it).
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → DUPLICATE
Comment on attachment 651735 [details] [diff] [review]
Patch

canceling review request per comment #2
Attachment #651735 - Flags: review?(surkov.alexander)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: