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

starboard/types.h: Replace kSbInt32Max with std::numeric_limits equivalents #4187

Merged
merged 2 commits into from
Oct 3, 2024

Conversation

yell0wd0g
Copy link
Contributor

As said this CL replaces kSbInt16Max with its std::numeric_limits<> equivalent, removes kSbInt32Min and kSbUInt32Max because they're unused, and uses the stdint.h INT32_MIN where appropriate (for kSbInvalidInt and in a C header file).

Verified via local x64x11 build.

[1] https://source.corp.google.com/search?q=kSb.%3FInt(32)&sq=&ss=h%2Flbshell-internal%2Fcobalt_src%2F%2B%2Frefs%2Fheads%2FCOBALT

b/366064934

…INT32_MIN where appropriate (C headers/files).
@datadog-cobalt-youtube
Copy link

datadog-cobalt-youtube bot commented Oct 2, 2024

Datadog Report

Branch report: b_366064934__C__fix_SbInt32s
Commit report: c9df14a
Test service: cobalt

✅ 0 Failed, 228913 Passed, 42 Skipped, 38m 16.37s Total Time

@yell0wd0g yell0wd0g marked this pull request as ready for review October 2, 2024 19:59
@yell0wd0g
Copy link
Contributor Author

@madhurajayaraman PTAL

@madhurajayaraman
Copy link
Contributor

Can you run on-device tests? (Right sidebar, add the label "on_device" and it should automatically kick off the tests)

@oxve oxve added the runtest Trigger presubmit workflows. label Oct 3, 2024
@github-actions github-actions bot removed the runtest Trigger presubmit workflows. label Oct 3, 2024
@oxve oxve enabled auto-merge (squash) October 3, 2024 02:52
@yell0wd0g
Copy link
Contributor Author

Thanks @oxve !

@oxve oxve merged commit 5bf159c into main Oct 3, 2024
61 of 62 checks passed
@oxve oxve deleted the b_366064934__C__fix_SbInt32s branch October 4, 2024 23:43
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.

3 participants