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
[corlib] Use String from CoreCLR #9134
Conversation
Assert.IsNotNull (ex.Message, "#A4"); | ||
Assert.AreEqual ("value", ex.ParamName, "#A5"); | ||
} | ||
Assert.AreEqual (string.Empty, new String ((sbyte*) null, 0, 0)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This means we are not expecting an ArgumentNullException
anymore, correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
correct
@@ -593,7 +593,8 @@ internal enum ExceptionArgument { | |||
byteOffset, | |||
minimumBufferSize, | |||
offset, | |||
values | |||
values, | |||
comparisonType |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You could have comparisonType,
to avoid the 2-line changes next time you need to add a value to this enum.
0ce3963
to
4befde8
Compare
4befde8
to
74e23f6
Compare
74e23f6
to
61ca696
Compare
@monojenkins commit apidiff and csproj |
(depends on fast-span)