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

Merge 'dotnet/master' into DefaultInterfaceImplementation #34044

Conversation

AlekseyTs
Copy link
Contributor

No description provided.

sharwell and others added 30 commits July 13, 2018 12:04
Add method to IsTypeAttributeContextBefore
Fix to Roslyn Style
Remove GetNextToken/GetPreviousToken
Remove white line
Fix wrong logic when is not Attribute List
…ons/RegexCharClass.cs

Co-Authored-By: CyrusNajmabadi <cyrus.najmabadi@gmail.com>
Neal Gafter and others added 19 commits March 11, 2019 10:24
…otnet#33892)

* Implement LDM decisions on the behavior of `??` and `??=` operators
Fixes dotnet#33743
Did some profiling and all of the slowness is due to peverify. It seems
to hit a wall about 700 levels of nesting. Once it's removed we can
easily test our tolerance levels in the compiler
handle extension methods in "Sync Namespace" refactoring
…tch expression arm (dotnet#33697)

* Correct parsing of a parenthesized constant pattern expression in switch expression arm
Fixes dotnet#33675
* Add a test that we properly bind and code-gen a parenthesized constant expression.
Nullable nontrivial constructor should propagate state of the argument to Value.
Fix stack overflow compiling deeply nested generic
Check nullness of a nullable receiver when invoking base methods.
Have to disable quiet restore explicitly on local dev machines when
doing a bootstrap build to work around this arcade issue:
dotnet/arcade#2220
Ref and out parameters with suppression should change the nullable state
for the argument being suppressed. Moreover, the changes should flow
through type inference.

Closes dotnet#27743
@AlekseyTs AlekseyTs requested review from a team as code owners March 12, 2019 18:41
@AlekseyTs AlekseyTs added Area-Compilers PR For Personal Review Only The PR doesn’t require anyone other than the developer to review it. New Language Feature - Default Interface Impl Default Interface Implementation and removed PR For Personal Review Only The PR doesn’t require anyone other than the developer to review it. labels Mar 12, 2019
@AlekseyTs
Copy link
Contributor Author

@dotnet/roslyn-compiler Need a sign-off for an integration PR.

Copy link
Member

@jcouv jcouv left a comment

Choose a reason for hiding this comment

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

Approving without detailed review (merging master into a feature branch)

@AlekseyTs AlekseyTs merged commit 92de39b into dotnet:features/DefaultInterfaceImplementation Mar 12, 2019
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.

None yet