Re: ANN: AdaYaml 0.1.0 (initial release)

[Felix Krause <contact@flyx.org>]

On 2017-08-19 15:28:37 +0000, Lucretia said:

> Just had a brief look at this. I disagree with having Text outside of 
> the Yaml package namespace, it might not be part of the spec, but it is 
> part of the API/project so should be inside as Yaml.Text. Also, if 
> there exists another package called Text somewhere, you will be 
> inflicting naming collisions.

Well, I thought about moving the Text stuff into a separate, 
independent project because I might want to use it somewhere else. For 
now, it exists as part of AdaYaml, but in the Text package. I also like 
having the Event type inside of the base Yaml package, which would not 
be possible if I moved Text to Yaml.Text because Event depends on Text. 
This is what made me believe Text is a very basic thing and should be 
separate from Yaml.

I am not aware of any publicly available libraries using Text as 
package name, but it does seem like a name that may lead to a 
collision. I might reconsider the package structure, but as soon as I 
start using the Text stuff elsewhere, I need to place it in a separate 
base package anyway. Perhaps I can come up with a better name to avoid 
collisions.

-- 
Regards,
Felix Krause