Comments on: swush-0.91

By: Omry

Omry — Mon, 27 Apr 2009 15:29:52 +0000

SamSol, good point.
although I think it does not change anything in regard to preserving comments.
you just preserve the multiline comments including whatever elements are in it.

By: SamSol

SamSol — Mon, 27 Apr 2009 11:49:52 +0000

Multiline comments also can be used to hide (disable) part of swush file.
Other way to disable part in swush file is renaming keys.
xmpp-client-config {
account {
server: server1
}
dont-start-this-account {
server: server2
}
}

By: Omry

Omry — Mon, 27 Apr 2009 11:44:00 +0000

artm, great question, and in fact I was just discussing this exact use case with SamSol which helps me on the java side and the language definition.
for now ignore them, but a future version will probably preserve them as you suggested.

By: artm

artm — Mon, 27 Apr 2009 11:40:19 +0000

are you really ignoring comments, Omry? if swush is to be used for configuration, you want to keep the comments intact. imagine, you load the config file, user changes some settings in gui, and you save the new config back – you want old comments be kept in logical places and all.

comments are either about what follows (comment starts on a new line) or what precedes (comment follows a variable : value pair) and should be stored in the memory representation “attached” appropriately, I think. then when you tweak documented settings they are stored documented.

By: Omry

Omry — Mon, 27 Apr 2009 08:54:25 +0000

Oops!
Thanks, fixed it.

By: Ali Anwar

Ali Anwar — Mon, 27 Apr 2009 08:49:25 +0000

The links are broken.

Thanks for the wonderful Idea!