Skip to content
This repository was archived by the owner on Dec 18, 2018. It is now read-only.

Faster CopyFrom(+Ascii)#527

Merged
halter73 merged 1 commit intoaspnet:devfrom
benaadams:copyfrom-perf
Jan 12, 2016
Merged

Faster CopyFrom(+Ascii)#527
halter73 merged 1 commit intoaspnet:devfrom
benaadams:copyfrom-perf

Conversation

@benaadams
Copy link
Contributor

From #519

Copy link
Member

Choose a reason for hiding this comment

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

Keep the ArraySegment<byte> overload? Are you just trying to avoid copies?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will put it back

@benaadams benaadams force-pushed the copyfrom-perf branch 2 times, most recently from 0bd4088 to 8bb9cea Compare December 25, 2015 00:24
@benaadams benaadams changed the title Faster CopyFrom Faster CopyFrom(+Ascii) Dec 25, 2015
@benaadams benaadams force-pushed the copyfrom-perf branch 2 times, most recently from b73dc03 to 3c38f48 Compare December 29, 2015 08:25
Copy link
Member

Choose a reason for hiding this comment

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

Any reason for removing these asserts?

@benaadams benaadams force-pushed the copyfrom-perf branch 2 times, most recently from 5fa84f4 to af593d2 Compare January 5, 2016 14:31
@halter73 halter73 merged commit caaf9d4 into aspnet:dev Jan 12, 2016
@benaadams benaadams deleted the copyfrom-perf branch May 10, 2016 11:05
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.

4 participants