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

Remove VM_IMAGE macros and IL2CPPMonoImage #735

Conversation

mderoy
Copy link

@mderoy mderoy commented Dec 6, 2017

No description provided.

@mderoy mderoy self-assigned this Dec 6, 2017
@@ -67,9 +67,8 @@
#define VM_ARRAY_GET_RANK(arr) il2cpp_array_rank(arr)
#define VM_ARRAY_BOUND_LENGTH(arr, i) il2cpp_array_bound_length(arr, i)
#define VM_ARRAY_BOUND_LOWER_BOUND(arr, i) il2cpp_array_bound_lower_bound(arr, i)
#define VM_IMAGE_GET_NAME(image) il2cpp_image_name(image)
//Fixme module name as image name seems bad

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the module and image always have the same name, so this might be OK.

@mderoy
Copy link
Author

mderoy commented Dec 7, 2017

merged in #746

@mderoy mderoy closed this Dec 7, 2017
@joncham joncham deleted the unity-master-staging-header-c-compatible-9 branch January 16, 2018 22:53
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.

3 participants