JosiahGoeman

Fix for XMLExporter issues in #2310

8

Includes fixes for all issues listed in https://github.com/jMonkeyEngine/jmonkeyengine/issues/2310 and refactors related classes for readability, adherence to style guide, and elimination of duplicate code. Also adds unit tests for implementations of...

BinaryExporter resets buffer positions to 0 when writing them.

2

When a buffer or buffer arraylist is written using BinaryExporter, [the buffer's position gets reset to zero.](https://github.com/jMonkeyEngine/jmonkeyengine/blob/db6353046cc34af7d09211860c37e4c4a722516f/jme3-core/src/plugins/java/com/jme3/export/binary/BinaryOutputCapsule.java#L955) XMLExporter handles this by [restoring the old position after writing.](https://github.com/jMonkeyEngine/jmonkeyengine/blob/db6353046cc34af7d09211860c37e4c4a722516f/jme3-plugins/src/xml/java/com/jme3/export/xml/DOMOutputCapsule.java#L856C9-L856C29) This seems easy...

XMLExporter has several bugs/inconsistencies with BinaryExporter

11

I've been struggling a bit with the XMLExporter recently in my game and this prompted me to write some unit tests for the JmeExporter/JmeImporter implementations. You can see the tests...