Skip to content

Use generic types in the ConvertRequest class#165

Merged
hinerm merged 4 commits into
masterfrom
convert-api
May 8, 2015
Merged

Use generic types in the ConvertRequest class#165
hinerm merged 4 commits into
masterfrom
convert-api

Conversation

@ctrueden

@ctrueden ctrueden commented May 8, 2015

Copy link
Copy Markdown
Member

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

hinerm commented May 8, 2015

Copy link
Copy Markdown
Member

🐙

@hinerm

hinerm commented May 8, 2015

Copy link
Copy Markdown
Member

(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