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

Update ChiselMemory.cs #328

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

FV7VR157
Copy link

@FV7VR157 FV7VR157 commented Aug 15, 2021

Removes CS0117 error in Unity 2021.2.0b7 " 'AllocatorManager.AllocatorHandle' does not contain a definition for 'Index' "
Not sure if it breaks anything in the process yet, I have no idea what I'm doing

Removes CS0117 error in Unity 2021.2.0b7 " 'AllocatorManager.AllocatorHandle' does not contain a definition for 'Index' "
Not sure if it breaks anything in the process, I have no idea what I'm doing
@LogicalError
Copy link
Member

LogicalError commented Aug 16, 2021

image

Thanks!
I'm not sure what happened, I think it's package manifest bullshit again .. but I suspect you're using an older package locally than I am, because when I look at the code for AllocatorHandle, then "Value" is read-only

I've updated the repository, making sure that all references to collections are set to the right package version, hopefully this will fix that for you. But this PR would break the project when using the newer collections package

@LogicalError LogicalError self-assigned this Aug 16, 2021
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.

None yet

2 participants