Skip to content

[runtime] The return value from mono_class_from_mono_type must be released. - #11729

Merged
rolfbjarne merged 1 commit into
dotnet:mainfrom
rolfbjarne:coreclr-release-return-value-mono-class-from-mono-type
May 31, 2021
Merged

[runtime] The return value from mono_class_from_mono_type must be released.#11729
rolfbjarne merged 1 commit into
dotnet:mainfrom
rolfbjarne:coreclr-release-return-value-mono-class-from-mono-type

Conversation

@rolfbjarne

Copy link
Copy Markdown
Member

Before:

There were 258042 MonoObjects created, 235166 MonoObjects freed, so 22876 were not freed. (dynamic registrar)
There were 205804 MonoObjects created, 204219 MonoObjects freed, so 1585 were not freed. (static registrar)

After:

There were 258050 MonoObjects created, 235177 MonoObjects freed, so 22873 were not freed. (dynamic registrar)
There were 205804 MonoObjects created, 204219 MonoObjects freed, so 1585 were not freed. (static registrar)

…eased.

Before:

    There were 258042 MonoObjects created, 235166 MonoObjects freed, so 22876 were not freed. (dynamic registrar)
    There were 205804 MonoObjects created, 204219 MonoObjects freed, so 1585 were not freed. (static registrar)

After:

    There were 258050 MonoObjects created, 235177 MonoObjects freed, so 22873 were not freed. (dynamic registrar)
    There were 205804 MonoObjects created, 204219 MonoObjects freed, so 1585 were not freed. (static registrar)
@rolfbjarne rolfbjarne added not-notes-worthy Ignore for release notes run-dotnet-tests Run all the .NET tests labels May 28, 2021
@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

🔥 Tests failed catastrophically on Build (no summary found). 🔥

Result file $(TEST_SUMMARY_PATH) not found.

GitHub pages

Results can be found in the following github pages (it might take some time to publish):

Pipeline on Agent
Merge e5acea5 into 165941e

@rolfbjarne

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

❌ [PR Build] Tests failed on Build ❌

Tests failed on Build.

API diff

✅ API Diff from stable

View API diff

API & Generator diff

API Diff (from PR only) (no change)
Generator Diff (no change)

GitHub pages

Results can be found in the following github pages (it might take some time to publish):

Test results

2 tests failed, 106 tests passed.

Failed tests

  • monotouch-test/Mac [dotnet]/Debug (CoreCLR) [dotnet]: Failed (Test run failed.
    Tests run: 2444 Passed: 2353 Inconclusive: 7 Failed: 4 Ignored: 87)
  • monotouch-test/Mac [dotnet]/Debug (CoreCLR, static registrar) [dotnet]: Failed (Test run failed.
    Tests run: 2441 Passed: 2353 Inconclusive: 7 Failed: 2 Ignored: 86)

Pipeline on Agent XAMBOT-1104.BigSur'
Merge e5acea5 into 51c9364

@rolfbjarne
rolfbjarne merged commit 6a83ed9 into dotnet:main May 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

not-notes-worthy Ignore for release notes run-dotnet-tests Run all the .NET tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants