Skip to content

Conversation

@prozolic
Copy link
Contributor

I have made the same changes to LongCount as to Count in v1.4.2.
Please check if it looks good.

@neuecc
Copy link
Member

neuecc commented May 1, 2025

Ok, thanks.

However, the optimization specifically for FromArray may lead to increased complexity and assembly size, which I want to minimize.
Since LongCount is used quite infrequently, I think it might not be an appropriate target for optimization.
That said, I don't think it makes much difference either way, so I'll add it.

@neuecc neuecc merged commit 4864243 into Cysharp:main May 1, 2025
4 checks passed
@prozolic prozolic deleted the LongCount branch May 1, 2025 11:50
@prozolic
Copy link
Contributor Author

prozolic commented May 1, 2025

Thanks for the merge.
Indeed, the LongCount is not something that's frequently used.

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