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

Side by Side Diff: runtime/bin/platform_linux.cc

Issue 8662006: Fix a number of issues with the process handling (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Addressed review comments from ager@ Created 9 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « runtime/bin/platform.h ('k') | runtime/bin/platform_macos.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2011, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2011, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 #include "bin/platform.h" 5 #include "bin/platform.h"
6 6
7 #include <signal.h>
7 #include <string.h> 8 #include <string.h>
8 #include <unistd.h> 9 #include <unistd.h>
9 10
10 11
12 bool Platform::Initialize() {
13 // Turn off the signal handler for SIGPIPE as it causes the process
14 // to terminate on writing to a closed pipe. Without the signal
15 // handler error EPIPE is set instead.
16 struct sigaction act;
17 bzero(&act, sizeof(act));
18 act.sa_handler = SIG_IGN;
19 if (sigaction(SIGPIPE, &act, 0) != 0) {
20 perror("Setting signal handler failed");
21 return false;
22 }
23 return true;
24 }
25
26
11 int Platform::NumberOfProcessors() { 27 int Platform::NumberOfProcessors() {
12 return sysconf(_SC_NPROCESSORS_ONLN); 28 return sysconf(_SC_NPROCESSORS_ONLN);
13 } 29 }
14 30
15 31
16 const char* Platform::OperatingSystem() { 32 const char* Platform::OperatingSystem() {
17 return "linux"; 33 return "linux";
18 } 34 }
19 35
20 36
21 char* Platform::StrError(int error_code) { 37 char* Platform::StrError(int error_code) {
22 static const int kBufferSize = 1024; 38 static const int kBufferSize = 1024;
23 char* error = static_cast<char*>(malloc(kBufferSize)); 39 char* error = static_cast<char*>(malloc(kBufferSize));
24 error[0] = '\0'; 40 error[0] = '\0';
25 char* error_str = strerror_r(error_code, error, kBufferSize); 41 char* error_str = strerror_r(error_code, error, kBufferSize);
26 if (error_str != error) { 42 if (error_str != error) {
27 size_t written = snprintf(error, kBufferSize, "%s", error_str); 43 size_t written = snprintf(error, kBufferSize, "%s", error_str);
28 ASSERT(written == strlen(error_str)); 44 ASSERT(written == strlen(error_str));
29 } 45 }
30 return error; 46 return error;
31 } 47 }
OLDNEW
« no previous file with comments | « runtime/bin/platform.h ('k') | runtime/bin/platform_macos.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698