Skip to content

Use generic types in the ConvertRequest class #165

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

Merged
merged 4 commits into from
May 8, 2015
Merged

Use generic types in the ConvertRequest class #165

merged 4 commits into from
May 8, 2015

Conversation

ctrueden
Copy link
Member

@ctrueden ctrueden commented May 8, 2015

A converter may need to verify type compatibility not just at the
Class level, but potentially for any generic type. Generic types are
not available at runtime from object instances, but they are
available in other scenarios; e.g., the ImageJ OPS framework uses the
converter framework to decide type compatibility (i.e., assignability
or convertibility) based on generic types. This is possible (rather
than generics being erased at runtime) because OPS parameters are
declared as instance fields, which can be introspected at runtime.

ctrueden added 4 commits May 7, 2015 11:50
A converter may need to verify type compatibility not just at the
Class level, but potentially for any generic type. Generic types are
not available at runtime from object instances, but they are
available in other scenarios; e.g., the ImageJ OPS framework uses the
converter framework to decide type compatibility (i.e., assignability
or convertibility) based on generic types. This is possible (rather
than generics being erased at runtime) because OPS parameters are
declared as instance fields, which can be introspected at runtime.
hinerm added a commit that referenced this pull request May 8, 2015
Use generic types in the ConvertRequest class
@hinerm hinerm merged commit fa24b15 into master May 8, 2015
@hinerm hinerm deleted the convert-api branch May 8, 2015 19:10
@hinerm
Copy link
Member

hinerm commented May 8, 2015

🐙

@hinerm
Copy link
Member

hinerm commented May 8, 2015

(translation: I require 8 arms to express my happiness with this PR)

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