Member net.sf.gridarta.gui.spells.SpellsUtils.readUntil (@NotNull final Reader stream, @NotNull final CharSequence tag, @Nullable final CharSequence abort)

Should the encounter of the abort string before the tag really be an EOFException? That's semantically wrong, but current usage code relies on this :(

Class net.sf.gridarta.gui.treasurelist.CFTreasureListTree

don't extend JTree, instead create a proper model

Member net.sf.gridarta.model.archetypeset.DefaultArchetypeSet< G extends GameObject< G, A, R, A extends MapArchObject< A, R extends Archetype< G, A, R >.archetypeMap

solve this issue

Member net.sf.gridarta.model.baseobject.BaseObject< G extends GameObject< G, A, R, A extends MapArchObject< A, R extends Archetype< G, A, R, T extends BaseObject< G, A, R, T >.getMultiRefCount ()

rename me, "Ref" is a term in Java short for "Reference" and this is not about "References" but about multi-square arches.

Class net.sf.gridarta.model.baseobject.GameObjectContainer< G extends GameObject< G, A, R, A extends MapArchObject< A, R extends Archetype< G, A, R >

In case of MapSquares, this class is most likely bogus regarding multi-part objects. This needs to be fixed.

Class net.sf.gridarta.model.baseobject.GameObjectContainer< G extends GameObject< G, A, R, A extends MapArchObject< A, R extends Archetype< G, A, R >.ReverseIterator< T extends GameObject<?, ?, ?>

move this class to JAPI

Class net.sf.gridarta.model.face.ArchFaceProvider

Move scanning for face files to this class, so the faces can be easily reloaded while the application is running.

Class net.sf.gridarta.model.face.CollectedFaceProvider

Construction of this class should succeed even in case the face file is unavailable.

Move parsing of the face files to this class, so the faces can be easily reloaded while the application is running.

Class net.sf.gridarta.model.face.DefaultFaceObject

think how this class could be serialized because size, offset etc. are not serializable.

Member net.sf.gridarta.model.io.AbstractGameObjectParser< G extends GameObject< G, A, R, A extends MapArchObject< A, R extends Archetype< G, A, R >.expandMulti (@NotNull final G gameObject, @NotNull final Collection< G > objects)

think whether expanding multi-parts here is a good idea, maybe MapModel is a better place.

Member net.sf.gridarta.model.io.AbstractGameObjectParser< G extends GameObject< G, A, R, A extends MapArchObject< A, R extends Archetype< G, A, R >.load (@NotNull final BufferedReader reader, @NotNull final String firstLine, @Nullable final Collection< G > objects)

Attributes not relevant during map parsing shouldn't be evaluated here, but in GameObject / ArchetypeParser instead.

Class net.sf.gridarta.model.io.PathManager

make more use of URI

Class net.sf.gridarta.model.mapmodel.InsertionMode

make replacement of multi-part objects work

Member net.sf.gridarta.model.validation.errors.ValidationError< G extends GameObject< G, A, R, A extends MapArchObject< A, R extends Archetype< G, A, R >.getMessage ()

validation errors should be able to provide their own arguments for message formatting

Class net.sf.gridarta.plugin.BshRunnable< G extends GameObject< G, A, R, A extends MapArchObject< A, R extends Archetype< G, A, R >

Document this class.

Class net.sf.gridarta.updater.Updater

move the updater to JAPI

Member net.sf.gridarta.utils.IOUtils.MAP_ENCODING

once the mailing list decided on the § character, eventually replace by "us-ascii".

Class net.sf.gridarta.var.atrinik.model.gameobject.GameObject

this class is not always the best place for multi-part object handling, see also net.sf.gridarta.model.baseobject.GameObjectContainer for issues about this

Class net.sf.gridarta.var.atrinik.model.maparchobject.MapArchObject

This class should be changed so map attributes are reflected in a more generic way like arch attributes.

Class net.sf.gridarta.var.crossfire.model.gameobject.GameObject

this class is not always the best place for multi-part object handling, see also net.sf.gridarta.model.baseobject.GameObjectContainer for issues about this

Class net.sf.gridarta.var.crossfire.model.maparchobject.MapArchObject

This class should be changed so map attributes are reflected in a more generic way like arch attributes.

Class net.sf.gridarta.var.daimonin.model.gameobject.GameObject

this class is not always the best place for multi-part object handling, see also net.sf.gridarta.model.baseobject.GameObjectContainer for issues about this

Class net.sf.gridarta.var.daimonin.model.maparchobject.MapArchObject

This class should be changed so map attributes are reflected in a more generic way like arch attributes.