Skip to content

[Runtime] Enable the -Wsign-conversion on trampolines-varargs files. - #7428

Merged
mandel-macaque merged 4 commits into
dotnet:masterfrom
mandel-macaque:sign-conversion-trampolines-varargs
Nov 14, 2019
Merged

[Runtime] Enable the -Wsign-conversion on trampolines-varargs files.#7428
mandel-macaque merged 4 commits into
dotnet:masterfrom
mandel-macaque:sign-conversion-trampolines-varargs

Conversation

@mandel-macaque

Copy link
Copy Markdown
Contributor

Enable the flag that was disabled via pragmas.

Continuation of PR: #7405

Enable the flag that was disabled via pragmas.

Continuation of PR: dotnet#7405
@mandel-macaque mandel-macaque changed the title [Runtime] Enable the -Wsign-conversion on trampilines-varargs files. [Runtime] Enable the -Wsign-conversion on trampolines-varargs files. Nov 13, 2019
Comment thread runtime/trampolines-varargs.h Outdated
Comment thread runtime/trampolines-varargs.m
@monojenkins

Copy link
Copy Markdown
Contributor

Build failure
Build succeeded
API Diff (from stable)
API Diff (from PR only) (no change)
Generator Diff (no change)
🔥 Test run failed 🔥

Test results

1 tests failed, 86 tests passed.

Failed tests

  • monotouch-test/watchOS 32-bits - simulator/Debug: Crashed

@monojenkins

Copy link
Copy Markdown
Contributor

Build failure
Build failed or was aborted

Build succeeded
API Diff (from stable)

@monojenkins

Copy link
Copy Markdown
Contributor

Build failure
Build succeeded
API Diff (from stable)
API Diff (from PR only) (no change)
Generator Diff (no change)
🔥 Test run failed 🔥

Test results

1 tests failed, 86 tests passed.

Failed tests

  • monotouch-test/watchOS 32-bits - simulator/Debug: Crashed

@mandel-macaque

Copy link
Copy Markdown
Contributor Author

@mandel-macaque
mandel-macaque merged commit 5e2231d into dotnet:master Nov 14, 2019
@mandel-macaque
mandel-macaque deleted the sign-conversion-trampolines-varargs branch November 14, 2019 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants