Summary: | On Solaris 10 with SS12 compiler, TCMalloc_PageHeap::runScavengerThread(void* context) is expected to return a value | ||
---|---|---|---|
Product: | WebKit | Reporter: | Ben Taylor <bentaylor.solx86> |
Component: | JavaScriptCore | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED FIXED | ||
Severity: | Normal | CC: | commit-queue, webkit.review.bot |
Priority: | P2 | ||
Version: | 528+ (Nightly build) | ||
Hardware: | PC | ||
OS: | Other | ||
Attachments: |
Description
Ben Taylor
2011-04-11 17:01:59 PDT
Created attachment 89138 [details]
Patch to fix compile issue with Solaris 10 and Studio 12 CC, this code expects to return a value.
Patch to fix the compile issue on Solaris 10 with Sun Studio 12 CC
Attachment 89138 [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/FastMalloc.cpp:1623: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Total errors found: 1 in 2 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 89141 [details]
Updated patch that fixes the currently broken white space issue.
Note to self. Simple fixes - run check-webkit-style. Missed the 2-space indent originally in the source.
Created attachment 89150 [details]
Updated patch that fixes the currently broken white space issue, and trailing )
Fixed now.
Created attachment 89152 [details]
patch that fixes the issues, and passes check-webkit-style locally
This should do it.
Comment on attachment 89152 [details]
patch that fixes the issues, and passes check-webkit-style locally
Honestly, I don't understand why we don't just always have this "return 0".
+#if (COMPILER(MSVC) || COMPILER(SUNCC))
Unneeded parentheses here.
The commit-queue encountered the following flaky tests while processing attachment 89152 [details]: java/lc3/JavaObject/JavaObjectToInt-002-n.html bug 58299 (author: ap@webkit.org) The commit-queue is continuing to process your patch. Comment on attachment 89152 [details] patch that fixes the issues, and passes check-webkit-style locally Clearing flags on attachment: 89152 Committed r83557: <http://trac.webkit.org/changeset/83557> All reviewed patches have been landed. Closing bug. |