soundasleep / iaml

Automatically exported from code.google.com/p/iaml
3 stars 1 forks source link

Replace isAssignableFrom with isInstance in codegen templates #107

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What new or enhanced feature are you proposing?
Search through Codegen templates and replace all instances of:
x.metaType.isAssignableFrom(model::Type)

To:
model::Type.isInstance(x)

What goal would this enhancement help you achieve?
This form is shorter and also more reliable; the first form will not match
subtypes. See http://journals.jevon.org/users/jevon-phd/entry/19734

Original issue reported on code.google.com by soundasleep on 28 Aug 2009 at 12:05

GoogleCodeExporter commented 9 years ago

Original comment by soundasleep on 4 Sep 2009 at 1:00

GoogleCodeExporter commented 9 years ago
Resolved in r1121.

Original comment by soundasleep on 9 Sep 2009 at 4:56

GoogleCodeExporter commented 9 years ago
Not all instances of 'metaType' have been replaced.

Original comment by soundasleep on 14 Sep 2009 at 10:35

GoogleCodeExporter commented 9 years ago

Original comment by soundasleep on 15 Sep 2009 at 5:40