Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expose ICellArray handle to extensions #2174

Closed
wants to merge 1 commit into from

Conversation

Kenzzer
Copy link
Member

@Kenzzer Kenzzer commented Jun 2, 2024

Resolves #720

Throughout my time using Sourcemod, I've come across multiple situations where being able to create a cell array handle and reading one would have been quite useful when designing natives for my extensions. Instead I've had to resort into creating a new handle type that mimics the behaviour of ArrayList, which is a source of many problems. It's extra work to put in place and unnecessary friction for plugin authors that need to read more documentation so they know how to loop through the pseudo-array like object. The current situation also adds an extra layer of complexity should the plugin author wants to make use of the ArrayList natives as they'd need to transpose the whole content from the extension's arraylike handle, to a new handle (arraylist) they created.

That feeling seems to have been echoed by other extensions authors in the past (see the linked issue), and still to this day I hear that desire from multiple places. I believe this is a justified addition to the extension API.

@Kenzzer Kenzzer closed this Jun 2, 2024
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.

Feature request: Expose ICellArray to Extensions
1 participant