FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

#2483: ChaCha20Poly1305 now copies the ciphertext into its decryption… by winne42 · Pull Request #2484 · bcgit/bc-java · GitHub

Repository navigation

#2483: ChaCha20Poly1305 now copies the ciphertext into its decryption… - #2484

Closed
winne42 wants to merge 1 commit into
bcgit:mainfrom
winne42:#2483-chacha20poly1305-decrypt-arraycopy
Closed

winne42 wants to merge 1 commit into
bcgit:mainfrom
winne42:#2483-chacha20poly1305-decrypt-arraycopy

Conversation

winne42 commented Oct 2, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

… buffer with System.arraycopy instead of one byte at a time
For details and benchmarks, see #2483.
This change is included in #2481 - only merge it if you decide against #2481.

AI Disclosure

Parts of this contribution were produced with the assistance of a generative AI tool (Claude Code), under the direction and review of the submitter, in line with the contributing guidelines. The submitter has reviewed and understands the code. To the best of his knowledge, the contribution does not reproduce third-party material.

winne42 marked this pull request as ready for review October 2, 2026 07:31
…ption buffer with System.arraycopy instead of one byte at a time
winne42 force-pushed the #2483-chacha20poly1305-decrypt-arraycopy branch from 2ab1e47 to 7697c2e Compare October 2, 2026 07:32
dghgit self-assigned this Oct 3, 2026

dghgit commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Thanks - superseded by #2481 (PR #2482), which is now merged.

dghgit closed this Oct 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL