Closed Bug 901187 Opened 9 years ago Closed 9 years ago
_stderr output to Output Debug String
+++ This bug was initially created as a clone of Bug #762519 +++ Similar to the bug I'm cloning, I would like printf_stderr to go to the Visual Studio console output to make debugging on Metro easier.
Comment on attachment 785331 [details] [diff] [review] Patch Not sure who should review this, feel free to bounce it as needed.
Sorry, haven't gotten the right patch generation flags on this windows machine.
Comment on attachment 785333 [details] [diff] [review] Patch Looks ok but I can't r+ something in xpcom glue.
Attachment #785333 - Flags: review?(jmathies) → review?(benjamin)
Comment on attachment 785333 [details] [diff] [review] Patch Is there a particular reason why you early-return here, instead of printing to both stderr and OutputDebugStringA ?
Not particularly, no. I can let it fall through. I can also refactor the code to reduce duplication but that might result in the stderr version getting truncated at 2047 bytes as well, so maybe we don't want that.
Yeah, I'm not particularly worried about the truncation, but I'd like it to always print to stderr, because I personally at least often pipe output to a file even while I'm in a debugger session.
Comment on attachment 785333 [details] [diff] [review] Patch r=me with the return removed
Attachment #785333 - Flags: review?(benjamin) → review+
Landed with the return statement removed: https://hg.mozilla.org/integration/mozilla-inbound/rev/e4a05768e8c3
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
You need to log in before you can comment on or make changes to this bug.