Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support <errno.h> across platforms #2648

Merged
merged 14 commits into from
Apr 13, 2024

Conversation

haozheng-cobalt
Copy link
Contributor

b/321999529

Change-Id: Id7bb43900ea7ded8e162be904b9fe9a0dee6fde6
Copy link

codecov bot commented Mar 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.76%. Comparing base (52dee1f) to head (265e189).
Report is 37 commits behind head on main.

❗ Current head 265e189 differs from pull request most recent head e0e4b0b. Consider uploading reports for the commit e0e4b0b to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2648      +/-   ##
==========================================
- Coverage   59.74%   56.76%   -2.99%     
==========================================
  Files        1825     1769      -56     
  Lines       88124    83058    -5066     
==========================================
- Hits        52652    47144    -5508     
- Misses      35472    35914     +442     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@haozheng-cobalt haozheng-cobalt force-pushed the adderrno branch 10 times, most recently from 7b90962 to a946d30 Compare March 29, 2024 22:28
Change-Id: Idb44248d7f5cfa8b54be581d2344b51ab533c9a9
@haozheng-cobalt haozheng-cobalt force-pushed the adderrno branch 2 times, most recently from 9624609 to 6188da1 Compare April 5, 2024 19:17
@haozheng-cobalt haozheng-cobalt force-pushed the adderrno branch 8 times, most recently from dd2ebee to a21682a Compare April 6, 2024 01:43
Change-Id: I717d4d35d0a299f1a9f86cd5e42429d753df3cfd
starboard/shared/win32/posix_emu/socket.cc Outdated Show resolved Hide resolved
starboard/shared/win32/posix_emu/socket.cc Outdated Show resolved Hide resolved
starboard/nplb/posix_compliance/posix_socket_errno_test.cc Outdated Show resolved Hide resolved
Change-Id: I0e26a80df79e43100b8e9635093beb6a8e9dbd5a
Change-Id: Ief096ab442426afcf2622b5d5920685b35469712
Change-Id: I2a983b9aa66b695e487f6f08149bd166fdd92691
Change-Id: I39807f23cc7bb6a4165480803e42be421c4b6382
Change-Id: I7819312eccf4bf5efdd0f33a95ebd819b82a25d0
Change-Id: Ice0334bed8df8e406d50a37089ce5c007a60858f
@haozheng-cobalt haozheng-cobalt merged commit 761065c into youtube:main Apr 13, 2024
377 of 381 checks passed
@haozheng-cobalt haozheng-cobalt deleted the adderrno branch April 13, 2024 21:18
niranjanyardi added a commit that referenced this pull request Apr 15, 2024
niranjanyardi added a commit that referenced this pull request Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants