small clean up of a11y logging

RESOLVED FIXED in mozilla15

Status

()

Core
Disability Access APIs
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: surkov, Assigned: surkov)

Tracking

unspecified
mozilla15
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Created attachment 628211 [details] [diff] [review]
patch

1) provide MsgBeing/End to allow logging without helper function like DocDestroy()
2) turn stacks logging into option
Attachment #628211 - Flags: review?(trev.saunders)
(Assignee)

Updated

5 years ago
Assignee: nobody → surkov.alexander
OS: Mac OS X → All
Hardware: x86 → All
Comment on attachment 628211 [details] [diff] [review]
patch

>+  vfprintf(stdout, aMsgText, argptr);

nit, why not use vprintf()?

>+ * Log the message ('title: text' format) on new line. Print the start and end
>+ * boundaries of the message body designed by '{' and '}' (2 spaces indent for

nit, designated by?
Attachment #628211 - Flags: review?(trev.saunders) → review+
(Assignee)

Comment 2

5 years ago
(In reply to Trevor Saunders (:tbsaunde) from comment #1)

> >+  vfprintf(stdout, aMsgText, argptr);
> 
> nit, why not use vprintf()?

ok
https://hg.mozilla.org/mozilla-central/rev/9a6e690f779f
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla15
You need to log in before you can comment on or make changes to this bug.