Skip to content

const for max buffer size for stackalloc#73

Merged
manigandham merged 1 commit intoullmark:masterfrom
Sa1Gur:master
May 15, 2022
Merged

const for max buffer size for stackalloc#73
manigandham merged 1 commit intoullmark:masterfrom
Sa1Gur:master

Conversation

@Sa1Gur
Copy link
Copy Markdown
Contributor

@Sa1Gur Sa1Gur commented May 15, 2022

Hi. I was interested in how does hashids calculated and spotted that constant for max buffer size allegeable for stackalloc could be introduced. I guess there is not reasons not to. May be I missed something.

introduced constant for max buffer size allegeable for stackalloc
@manigandham
Copy link
Copy Markdown
Collaborator

This is fine. 512 is a little big for stack allocation but you're right it's better as a const.

@manigandham manigandham merged commit eb3ca5b into ullmark:master May 15, 2022
@manigandham manigandham self-assigned this May 22, 2022
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