Skip to content

Conversation

@cston
Copy link
Contributor

@cston cston commented Nov 16, 2015

No description provided.

@cston
Copy link
Contributor Author

cston commented Nov 16, 2015

@amcasey, @KevinH-MS

Copy link
Member

Choose a reason for hiding this comment

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

What does this test? Did it used to throw?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The test would have resulted in an exception back when the bug was logged, but there have been significant changes to emit since then so that code path is no longer hit. But it seemed like an interesting test case.

@amcasey
Copy link
Member

amcasey commented Nov 16, 2015

👍

cston added a commit that referenced this pull request Nov 16, 2015
Add a couple of tests and remove unused EE source file
@cston cston merged commit 80d3bb7 into dotnet:master Nov 16, 2015
@cston cston deleted the bugs branch November 16, 2015 20:00
@KevinH-MS
Copy link
Contributor

In late, but 👍

github-actions bot pushed a commit that referenced this pull request Apr 1, 2025
* tests + unshipped markdown

* using Symbol comparison

* warning message title

* Unshipped.md update and cleanup

* avoid emitting warning in case of limit disabling

* supporting SocketsHttpHandler

* Binary operation fix

---------

Co-authored-by: Amir Solhi <>
Co-authored-by: Miha Zupan <mihazupan.zupan1@gmail.com>
Co-authored-by: Buyaa Namnan <buyankhishig.namnan@microsoft.com>
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.

4 participants