Bug 72970 - Add WTF infrastructure for the BlackBerry port
Summary: Add WTF infrastructure for the BlackBerry port
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Daniel Bates
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-22 10:41 PST by Daniel Bates
Modified: 2011-11-23 10:19 PST (History)
4 users (show)

See Also:


Attachments
Patch (9.71 KB, patch)
2011-11-22 10:58 PST, Daniel Bates
tonikitoo: review+
dbates: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Bates 2011-11-22 10:41:12 PST
Towards getting JavaScriptCore building for the BlackBerry port we need to add some BlackBerry-specific code for threading, assertions, random number seed generation, and strings.

We also need to add a some quirks to wtf/MathExtras.h due to differences between the GNU and QNX C++ headers. We'll look to follow up and remove these quirks.
Comment 1 Daniel Bates 2011-11-22 10:42:02 PST
PR 118379
Comment 2 Daniel Bates 2011-11-22 10:58:45 PST
Created attachment 116250 [details]
Patch
Comment 3 WebKit Review Bot 2011-11-22 11:04:01 PST
Attachment 116250 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/ChangeLog', u'Source..." exit_code: 1

Source/JavaScriptCore/wtf/text/WTFString.h:52:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
Total errors found: 1 in 8 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 4 Daniel Bates 2011-11-22 11:10:06 PST
(In reply to comment #3)
> Attachment 116250 [details] did not pass style-queue:
> [...]
> Source/JavaScriptCore/wtf/text/WTFString.h:52:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
> Total errors found: 1 in 8 files

From my understanding of reading (3) of section Indentation on <http://www.webkit.org/coding/coding-style.html>, the contents of a nested namespace should be indented.
Comment 5 David Levin 2011-11-22 11:15:27 PST
(In reply to comment #4)
> (In reply to comment #3)
> > Attachment 116250 [details] [details] did not pass style-queue:
> > [...]
> > Source/JavaScriptCore/wtf/text/WTFString.h:52:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
> > Total errors found: 1 in 8 files
> 
> From my understanding of reading (3) of section Indentation on <http://www.webkit.org/coding/coding-style.html>, the contents of a nested namespace should be indented.

Sounds right. No one has implemented this subcase of the check yet.
Comment 6 Daniel Bates 2011-11-22 11:17:31 PST
Comment on attachment 116250 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=116250&action=review

> Source/JavaScriptCore/wtf/RandomNumberSeed.h:59
> +#elif OS(UNIX) && !PLATFORM(BLACKBERRY)

I spoke with Antonio Gomes on IRC about this change. We may be able to use the UNIX code path. I'll look into this some more and follow up if needed.
Comment 7 Daniel Bates 2011-11-22 19:30:49 PST
Committed r101041: <http://trac.webkit.org/changeset/101041>
Comment 8 Antonio Gomes 2011-11-23 10:19:20 PST
(In reply to comment #6)
> (From update of attachment 116250 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=116250&action=review
> 
> > Source/JavaScriptCore/wtf/RandomNumberSeed.h:59
> > +#elif OS(UNIX) && !PLATFORM(BLACKBERRY)
> 
> I spoke with Antonio Gomes on IRC about this change. We may be able to use the UNIX code path. I'll look into this some more and follow up if needed.

filed bug 73028 about it.