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

Issue 21610003: Prepared newlib toolchain for socket implementation (Closed)

Created:
7 years, 4 months ago by torinmr
Modified:
7 years, 4 months ago
Reviewers:
noelallen1, bradn, Sam Clegg
CC:
chromium-reviews, binji, Sam Clegg
Base URL:
https://chromium.googlesource.com/chromium/src.git@headers
Visibility:
Public.

Description

Prepared newlib toolchain for socket implementation Now that appropriate socket headers have been found for newlib, two things need to be done before socket implementation work can begin. First, wrappers must be added to kernel_wrap_newlib. Second, some helper functions provided by glibc need to be implemented. This CL does both. In particular, inet_ntoa() and inet_ntop() have been implemented, allowing the conversion of IPv4 and IPv6 addresses to strings. NOTRY=true BUG=None TEST=None R=noelallen@chromium.org,bradnelson@google.com Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=215937

Patch Set 1 : #

Total comments: 9

Patch Set 2 : minor fixes #

Patch Set 3 : #

Patch Set 4 : #

Patch Set 5 : #

Patch Set 6 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+251 lines, -7 lines) Patch
M native_client_sdk/src/build_tools/sdk_files.list View 1 2 3 4 5 1 chunk +1 line, -0 lines 0 comments Download
A native_client_sdk/src/libraries/nacl_io/inet_ntoa.cc View 1 2 3 4 1 chunk +65 lines, -0 lines 0 comments Download
M native_client_sdk/src/libraries/nacl_io/kernel_wrap_newlib.cc View 1 2 3 4 5 5 chunks +76 lines, -6 lines 0 comments Download
M native_client_sdk/src/libraries/nacl_io/library.dsc View 1 2 3 4 5 1 chunk +1 line, -0 lines 0 comments Download
M native_client_sdk/src/libraries/nacl_io/ossocket.h View 1 chunk +1 line, -1 line 0 comments Download
M native_client_sdk/src/libraries/nacl_io_test/socket_test.cc View 1 1 chunk +107 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
torinmr
7 years, 4 months ago (2013-08-01 20:04:38 UTC) #1
Sam Clegg
https://chromiumcodereview.appspot.com/21610003/diff/3001/native_client_sdk/src/libraries/nacl_io/inet_ntoa.cc File native_client_sdk/src/libraries/nacl_io/inet_ntoa.cc (right): https://chromiumcodereview.appspot.com/21610003/diff/3001/native_client_sdk/src/libraries/nacl_io/inet_ntoa.cc#newcode1 native_client_sdk/src/libraries/nacl_io/inet_ntoa.cc:1: #include <arpa/inet.h> Missing copyright. https://chromiumcodereview.appspot.com/21610003/diff/3001/native_client_sdk/src/libraries/nacl_io/inet_ntoa.cc#newcode12 native_client_sdk/src/libraries/nacl_io/inet_ntoa.cc:12: static inline uint8_t ...
7 years, 4 months ago (2013-08-01 20:50:53 UTC) #2
torinmr
https://chromiumcodereview.appspot.com/21610003/diff/3001/native_client_sdk/src/libraries/nacl_io/inet_ntoa.cc File native_client_sdk/src/libraries/nacl_io/inet_ntoa.cc (right): https://chromiumcodereview.appspot.com/21610003/diff/3001/native_client_sdk/src/libraries/nacl_io/inet_ntoa.cc#newcode1 native_client_sdk/src/libraries/nacl_io/inet_ntoa.cc:1: #include <arpa/inet.h> On 2013/08/01 20:50:53, Sam Clegg wrote: > ...
7 years, 4 months ago (2013-08-01 22:04:00 UTC) #3
Sam Clegg
lgtm
7 years, 4 months ago (2013-08-06 00:33:40 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/torinmr@chromium.org/21610003/39001
7 years, 4 months ago (2013-08-06 18:42:30 UTC) #5
commit-bot: I haz the power
7 years, 4 months ago (2013-08-06 18:43:14 UTC) #6
Message was sent while issue was closed.
Change committed as 215937

Powered by Google App Engine
This is Rietveld 408576698