Skip to content
This repository was archived by the owner on Nov 1, 2020. It is now read-only.

Merge master to nmirror - #5046

Merged
jkotas merged 21 commits into
nmirrorfrom
master
Dec 1, 2017
Merged

Merge master to nmirror#5046
jkotas merged 21 commits into
nmirrorfrom
master

Conversation

@jkotas

@jkotas jkotas commented Dec 1, 2017

Copy link
Copy Markdown
Member

No description provided.

jkotas and others added 21 commits November 25, 2017 08:34
)

Avoids unnecessary allocations and copying, and reduces the amount of unsafe code.

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
This PR enforces the rule that methods with `[NativeCallable]` attribute can only be called from unmanaged code. A stub that throws a `InvalidProgramException` is put in place of a call from managed (C#) code

Fixes #5012
Test.CoreLib has bitrotted again.
It's more convenient to just let the last one win.
* Make double-clicking item open item view
* Reorder list box and button
* Add UAC manifest
* Added additional tests for delegate assignment/return.

* Added ftn/delegate test cases for byte/int parameters and enums.
…#15238)

As part of this, I needed to fix-up several functions that parsed strings assuming they end with '\0', which no longer holds once we switch to using spans.

Also:
- Switched various call sites to using a cast rather than AsReadOnlySpan.
- Updated string.Format / StringBuilder.AppendFormat to use the new span-based support to avoid substring allocations.
- Removed some TODOs that were left in the codebase from some of my previous changes

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Signed-off-by: Petr Bred <bredpetr@gmail.com>
* enabled thread static fields, changed vtable dispatch to not be dependent on EETypeNode, added typed node factory
…rmat

Add double/float.TryFormat

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
@jkotas
jkotas merged commit fc4b463 into nmirror Dec 1, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants