- 
          
- 
                Notifications
    You must be signed in to change notification settings 
- Fork 1.2k
MorphControl: serialize target list & add minor cleanups #1553
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
Conversation
…() & add minor cleanups.
| I believe it's unnecessary to serialize the list of targets, since the list will get regenerated by  | 
| 
 I was thinking like so but found out setSpatial() is not called when loading from serialized file. You can confirm that by running TestMorph (which uses BinaryExporter.saveAndLoad). Without serialize the list it will throw or by running TestAnimMorphSerialization. | 
| Thanks for your latest contribution to JMonkeyEngine! | 
* MorphControl: move target locating out of update loop into setSpatial() & add minor cleanups. * Serialize target list * Add javadoc * Fix comment.
| I appreciate you for reviewing and merging the PR. :) | 
See:
https://hub.jmonkeyengine.org/t/morphcontrol-question/44599?u=ali_rs & #1548 (comment)