Bug 29808 - [Qt] assertions to be visible on Symbian
Summary: [Qt] assertions to be visible on Symbian
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: S60 Hardware S60 3rd edition
: P3 Normal
Assignee: Nobody
URL:
Keywords: Qt
Depends on:
Blocks: 27065
  Show dependency treegraph
 
Reported: 2009-09-28 08:32 PDT by Janne Koskinen
Modified: 2009-09-29 09:10 PDT (History)
2 users (show)

See Also:


Attachments
vprintf patch (1.09 KB, patch)
2009-09-28 08:32 PDT, Janne Koskinen
ddkilzer: review-
Details | Formatted Diff | Diff
vprintf patch second attempt (1.09 KB, patch)
2009-09-29 00:38 PDT, Janne Koskinen
no flags Details | Formatted Diff | Diff
vprintf patch third attempt (1.14 KB, patch)
2009-09-29 00:48 PDT, Janne Koskinen
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Janne Koskinen 2009-09-28 08:32:33 PDT
Created attachment 40236 [details]
vprintf patch

Qt port of Symbian uses Open C to access posix libraries.
assertions.h uses vprintf to print out the assert messages into stderr.
In Open C stderr prints are lost somewhere and only stdout can be seen from
log files. This change that makes assert messages to be visible on Symbian.
Comment 1 David Kilzer (:ddkilzer) 2009-09-28 16:35:00 PDT
Comment on attachment 40236 [details]
vprintf patch

> diff --git a/JavaScriptCore/ChangeLog b/JavaScriptCore/ChangeLog
> index f6a644a..894ff9a 100644
> --- a/JavaScriptCore/ChangeLog
> +++ b/JavaScriptCore/ChangeLog
> @@ -1,3 +1,15 @@
> +2009-09-28  Janne Koskinen  <janne.p.koskinen@digia.com>
> +
> +        Reviewed by NOBODY (OOPS!).
> +
> +        [Qt] Assert messages prints visible in Symbian
> +
> +        Asserts use vprintf to print the messages to stderr.
> +        In Symbian Open C it is not possible to see stderr so
> +        I routed the messages to stdout instead.
> +
> +        * wtf/Assertions.cpp:
> +

Please include a reference to the bug URL in the ChangeLog entry.

> +#ifdef PLATFORM(SYMBIAN)

This should be:

#if PLATFORM(SYMBIAN)

r- to address the above issues.
Comment 2 Janne Koskinen 2009-09-29 00:38:50 PDT
Created attachment 40286 [details]
vprintf patch second attempt

Corrected patch based on review comments.
Comment 3 Janne Koskinen 2009-09-29 00:48:38 PDT
Created attachment 40287 [details]
vprintf patch third attempt

Corrected based on review comments.
Added bug url and changed to use #if instead of #ifdef.
Comment 4 David Kilzer (:ddkilzer) 2009-09-29 09:01:23 PDT
Comment on attachment 40287 [details]
vprintf patch third attempt

r=me
Comment 5 WebKit Commit Bot 2009-09-29 09:10:30 PDT
Comment on attachment 40287 [details]
vprintf patch third attempt

Clearing flags on attachment: 40287

Committed r48878: <http://trac.webkit.org/changeset/48878>
Comment 6 WebKit Commit Bot 2009-09-29 09:10:45 PDT
All reviewed patches have been landed.  Closing bug.