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

Jump to bottom

Need a better validation when there are native fields in a class (direct or from the super class chain) #12455

Closed
a-siva opened this issue Aug 14, 2013 · 2 comments
Closed

Need a better validation when there are native fields in a class (direct or from the super class chain) #12455

a-siva opened this issue Aug 14, 2013 · 2 comments
Assignees
@a-siva
Labels
area-vm Use area-vm for VM related issues, including code coverage, FFI, and the AOT and JIT backends. type-code-health Internal changes to our tools and workflows to make them cleaner, simpler, or more maintainable

Comments

@a-siva
Copy link
Contributor

a-siva commented Aug 14, 2013

Currently the VM has a check to ensure that there is a native resolver in the library in which classes with native fields are defined. This is a
bit restrictive and we need a better validation scheme.

Maybe we don't predefine NativeFieldWrapper classes and instead have them
be defined by the embedder, the burden of validating then falls on the
embedder.
@iposva-google
Copy link
Contributor

I agree that these predefined native wrappers might have outlived their purpose.

Added Accepted label.

@a-siva a-siva added Type-Defect area-vm Use area-vm for VM related issues, including code coverage, FFI, and the AOT and JIT backends. labels Oct 21, 2013
@a-siva a-siva self-assigned this Oct 21, 2013
@kevmoo kevmoo added type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) and removed priority-unassigned labels Feb 29, 2016
@zanderso zanderso added type-code-health Internal changes to our tools and workflows to make them cleaner, simpler, or more maintainable and removed type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) labels Jan 5, 2017
@mraleph
Copy link
Member

mraleph commented Mar 12, 2018

Nothing is happening since 2013. Please reopen @a-siva if you consider working on this any time soon.

@mraleph mraleph closed this as completed Mar 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@a-siva a-siva

Labels
area-vm Use area-vm for VM related issues, including code coverage, FFI, and the AOT and JIT backends. type-code-health Internal changes to our tools and workflows to make them cleaner, simpler, or more maintainable
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@kevmoo @mraleph @iposva-google @zanderso @a-siva