Skip to content
This repository has been archived by the owner on Nov 5, 2019. It is now read-only.

Include checksum in generated class names (to resolve #51) #53

Closed
cowtowncoder opened this issue Mar 3, 2015 · 0 comments
Closed

Include checksum in generated class names (to resolve #51) #53

cowtowncoder opened this issue Mar 3, 2015 · 0 comments
Milestone

Comments

@cowtowncoder
Copy link
Member

(see #51 for background)

As things are, generated classes use simple naming scheme, which works ok with one notable exception: when properties are included/excluded with annotations, resulting accessor classes will differ.
But since it is not possible re-load classes, the initial definition remains, and may cause exception (or worse, potentially wrong output or binding).

One simple thing to do is to calculate checksum on bytecode and attach that as part of classname. This should result in different class name if actual contents change.

If even this fails we may need to consider use of randomized id instead; but let's try this first.

@cowtowncoder cowtowncoder added this to the 2.6.0 milestone Mar 4, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant