Skip to content

Conversation

@kzrnm
Copy link
Contributor

@kzrnm kzrnm commented Jan 10, 2024

Fix #96679 and Fix #94610

@ghost ghost added area-System.Numerics community-contribution Indicates that the PR has been added by a community member labels Jan 10, 2024
@ghost
Copy link

ghost commented Jan 10, 2024

Tagging subscribers to this area: @dotnet/area-system-numerics
See info in area-owners.md if you want to be subscribed.

Issue Details

Fix #94610

the pull request contains #96538 and #96680.

Author: kzrnm
Assignees: -
Labels:

area-System.Numerics

Milestone: -

@huoyaoyuan
Copy link
Member

Bringing commits from other pull requests may confuse reviewers.

@kzrnm
Copy link
Contributor Author

kzrnm commented Jan 10, 2024

@huoyaoyuan Noted. I'll change this pull request to draft until the preceding pull requests are resolved.

@kzrnm kzrnm marked this pull request as draft January 10, 2024 10:18
@tannergooding
Copy link
Member

The other PR is merged now

kzrnm added 5 commits January 11, 2024 09:06
Use typeof and nameof directly.
Use typeof and nameof as in BigNumberTools.
@kzrnm kzrnm force-pushed the fix/BigIntegerRegression branch from 6e81ea3 to 5407108 Compare January 11, 2024 00:06
@kzrnm kzrnm marked this pull request as ready for review January 11, 2024 00:07
@tannergooding tannergooding merged commit 6508018 into dotnet:main Jan 12, 2024
@kzrnm kzrnm deleted the fix/BigIntegerRegression branch January 12, 2024 18:35
@adamsitnik adamsitnik added this to the 9.0.0 milestone Jan 15, 2024
tmds pushed a commit to tmds/runtime that referenced this pull request Jan 23, 2024
…t#96744)

* Fix BigNumberTools

Use typeof and nameof directly.

* Fix BigIntTools

Use typeof and nameof as in BigNumberTools.

* RunWithFakeThreshold hack

* Add regression test

* Fix assertion
@github-actions github-actions bot locked and limited conversation to collaborators Feb 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-System.Numerics community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Some tests in parseTest are never run. System numerics abort at System.Numerics.BigIntegerCalculator.Multiply

4 participants