From: pondlife <pondlife_at_ntlworld.com>
Date: Wed, 13 Feb 2008 08:57:50 -0000

Hi Jonathan,

The H300 file isn't valid; I didn't look at the others. It looks like
you're missing some </setting> tags.

Would it perhaps make sense to group by type? e.g.

<settings>
   <type name = "sound">
      <setting name="volume"/>
   </type>
</settings>

Also, I've always tried to avoid use of attributes, but that's probably just
me having an irrational preferance for tags over attributes. e.g.

<Settings>
  <Type name="sound">
    <Volume min="-60" max="0" default="0"/>
    <Bass min="0" max="30" default="0"/>
    <Channels default="stereo">
      <option>stereo</option>
      <option>mono</option>
      <option>custom</option>
      <option>mono left</option>
      <option>mono right</option>
      <option>karaoke</option>
    </Channels>
  </Type>
</Settings>

I just find that more human-readable, which might not be relevant.

Just some ideas...

pondlife
Received on 2008-02-13