Class for XML configuration not placed in the correct module.
Description
Environment
Activity

Magno Nascimento October 25, 2018 at 2:39 PMEdited
I see, okay then, I'll mark it as CLOSED
. Thank you for the explanation!

Garret Wilson October 25, 2018 at 2:34 PM
No, the Properties
class is something sort of special. It exists as part of Java, and it's somewhat separate from its file format (even though they built serialization into the class, which wasn't a good idea—it's an old class).
Put another way, we agree that config-file-format-properties
should be a separate module for the format. But for the configuration that uses Properties
, I integrated it into the main configuration class because Properties
is such an integral part of the Java standard library. It comes built into Java.

Magno Nascimento October 25, 2018 at 2:27 PM
And what about config-properties
? It seems the same thing as the XML stuff.

Garret Wilson October 25, 2018 at 2:25 PM
Ah, I see. But in this case I don't think you need to do that. The only thing using XmlConfiguration
is the XML file format, and we can always refactor it in the future if we need it—but I don't think we'll need it.
The reason I created a separate config-urf
submodule with UrfConfiguration
is that there will be several formats (TURF, CURF, 3URF, etc) that all use an UrfConfiguration
because they will parse to an URF graph. But I don't think that applies to the XML situation.

Magno Nascimento October 25, 2018 at 2:17 PM
I'm saying that instead of creating three submodules:
I created only two:
Details
Assignee
Magno NascimentoMagno Nascimento(Deactivated)Reporter
Magno NascimentoMagno Nascimento(Deactivated)Priority
Major
Details
Details
Assignee

Reporter

In CONFOUND-2, the classes
XmlConfiguration
andXmlConfigurationFileFormat
are in the submoduleconfig-file-format-xml
, when, to be consistent, it should have the classXmlConfiguration
on another new submodule calledconfig-xml
.