Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(111)

Issue 653263003: Fix GetCurrentThreadId for Solaris etc.. (Closed)

Created:
6 years, 2 months ago by jochen (gone - plz use gerrit)
Modified:
6 years ago
Reviewers:
Sven Panne
CC:
v8-dev
Project:
v8
Visibility:
Public.

Description

Fix GetCurrentThreadId for Solaris etc.. BUG=v8:3620 R=svenpanne@chromium.org LOG=n Committed: https://code.google.com/p/v8/source/detail?r=24634

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M src/base/platform/platform-posix.cc View 1 chunk +1 line, -1 line 1 comment Download

Messages

Total messages: 5 (0 generated)
jochen (gone - plz use gerrit)
6 years, 2 months ago (2014-10-15 08:44:40 UTC) #1
Sven Panne
LGTM (almost :-} ) https://codereview.chromium.org/653263003/diff/1/src/base/platform/platform-posix.cc File src/base/platform/platform-posix.cc (right): https://codereview.chromium.org/653263003/diff/1/src/base/platform/platform-posix.cc#newcode263 src/base/platform/platform-posix.cc:263: return static_cast<int>(pthread_self()); Hmmm, this is ...
6 years, 2 months ago (2014-10-15 08:54:52 UTC) #2
jochen (gone - plz use gerrit)
Committed patchset #1 (id:1) manually as 24634 (presubmit successful).
6 years, 2 months ago (2014-10-15 12:39:02 UTC) #3
Sven Panne
On 2014/10/15 08:54:52, Sven Panne wrote: > [...] This needs to be fixed for real ...
6 years, 1 month ago (2014-10-27 07:34:58 UTC) #4
Robert Nagy
6 years ago (2014-12-07 13:04:12 UTC) #5
Message was sent while issue was closed.
On 2014/10/27 07:34:58, Sven Panne wrote:
> On 2014/10/15 08:54:52, Sven Panne wrote:
> > [...] This needs to be fixed for real at some point, but let's see what
breaks
> for
> > now... :-/
> 
> Fun fact: This is one of the reasons our NaCL builds fail:
>
http://chromegw.corp.google.com/i/client.v8/builders/NaCl%20V8%20Linux64%20-%...


OpenBSD breaks with this change.

../../v8/src/base/platform/platform-posix.cc:263:10: error: static_cast from
'pthread_t' (aka 'pthread *') to 'int' is not allowed
  return static_cast<int>(pthread_self());
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Powered by Google App Engine
This is Rietveld 408576698