Skip to content

[Runtime] Enable the -Wsign-conversion on monotouch-debug files. - #7459

Merged
mandel-macaque merged 4 commits into
dotnet:masterfrom
mandel-macaque:sign-conversion-monotouch-debug.m
Nov 20, 2019
Merged

[Runtime] Enable the -Wsign-conversion on monotouch-debug files.#7459
mandel-macaque merged 4 commits into
dotnet:masterfrom
mandel-macaque:sign-conversion-monotouch-debug.m

Conversation

@mandel-macaque

Copy link
Copy Markdown
Contributor

Enable the flag that was disabled via pragmas and fix warnings.

Continuation of PR: #7405

Enable the flag that was disabled via pragmas and fix warnings.

Continuation of PR: dotnet#7405
@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

@dalexsoto dalexsoto left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did not know about ssize_t 👍

Comment thread runtime/monotouch-debug.m Outdated
Comment thread runtime/monotouch-debug.m Outdated
Comment thread runtime/monotouch-debug.m Outdated
@monojenkins

Copy link
Copy Markdown
Contributor

Build failure
Build failed or was aborted

@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

2 tests failed, 85 tests passed.

Failed tests

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

Comment thread runtime/monotouch-debug.m Outdated
Co-Authored-By: Rolf Bjarne Kvinge <rolf@xamarin.com>
@mandel-macaque

Copy link
Copy Markdown
Contributor Author

@mandel-macaque
mandel-macaque merged commit 8f6a10f into dotnet:master Nov 20, 2019
@mandel-macaque
mandel-macaque deleted the sign-conversion-monotouch-debug.m branch November 20, 2019 18:42
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.

5 participants