Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Omit calls to CCCryptorFinal#70618

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
vcsjones merged 2 commits intodotnet:mainfromvcsjones:no-cc-final
Jun 12, 2022
Merged

Conversation

vcsjones
Copy link
Member

@vcsjonesvcsjones commentedJun 11, 2022•
edited
Loading

Our use of CommonCrypto permits us to omit calls to CCCryptorFinal (see comments added in PR), which gives a minor performance improvement.

The performance improvement will be about 5%-7% when operating on a single block of data to transform.

Our use of CommonCrypto permits us to omit calls CCCryptorFinal, which gives a minor performance improvement.
@ghost
Copy link

Tagging subscribers to this area: @dotnet/area-system-security,@vcsjones
See info inarea-owners.md if you want to be subscribed.

Issue Details

Our use of CommonCrypto permits us to omit calls CCCryptorFinal (see comments added in PR), which gives a minor performance improvement.

The performance improvement will be about 5%-7% when operating on a single block of data to transform.

Author:vcsjones
Assignees:vcsjones
Labels:

area-System.Security

Milestone:-

…/Cryptography/AppleCCCryptorLite.csCo-authored-by: Stephen Toub <stoub@microsoft.com>
@vcsjones
Copy link
MemberAuthor

S.IO.FileSystem.Tests failures are#70626. Merging.

@vcsjonesvcsjones merged commitffff070 intodotnet:mainJun 12, 2022
@vcsjonesvcsjones deleted the no-cc-final branchJune 12, 2022 21:34
@ghostghost locked asresolvedand limited conversation to collaboratorsAug 6, 2022
@bartonjsbartonjs added the needs-further-triageIssue has been initially triaged, but needs deeper consideration or reconsideration labelAug 22, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.
Reviewers

@stephentoubstephentoubstephentoub approved these changes

@bartonjsbartonjsbartonjs approved these changes

Assignees

@vcsjonesvcsjones

Labels
area-System.Securityneeds-further-triageIssue has been initially triaged, but needs deeper consideration or reconsideration
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@vcsjones@stephentoub@bartonjs
[8]ページ先頭
©2009-2025 Movatter.jp