Update the format of default.xml to allow override.

Update the format of default.xml to allow override.

This patch allows value inheritance from the previous matches.
With the new format:

<engines>
  <engine>
    <name>*</name>
    <rank>0</rank>
    <preedit-highlight>FALSE</preedit-highlight>
  </engine>
  <engine>
    <name>m17n:ja:*</name>
    <rank>2</rank>
    <preedit-highlight>TRUE</preedit-highlight>
  </engine>
  <engine>
    <name>m17n:ja:anthy</name>
    <rank>1</rank>
  </engine>
</engines>

We will get preedit-highlight == TRUE and rank == 1 for m17n:ja:anthy.

BUG=none
TEST=manual

[Daiki Ueno, 2011-09-02 06:52:18]

In Fedora, we have several options defined in default.xml.  However, the current
default.xml is not so efficient that those options can be maintained easily,
because every engine shall match with only one <engine> element.

This patch allows value override like fontconfig configuration files.

[Daiki Ueno, 2011-09-02 06:55:40]

On 2011/09/02 06:52:18, Daiki Ueno wrote:
> In Fedora, we have several options defined in default.xml.  However, the
current
> default.xml is not so efficient that those options can be maintained easily,
> because every engine shall match with only one <engine> element.
> 
> This patch allows value override like fontconfig configuration files.

For example, this patch reduces the amount of default.xml.in.in in the following
patch:

http://pkgs.fedoraproject.org/gitweb/?p=ibus-m17n.git;a=blob;f=ibus-m17n-xkb-...

to:

http://pkgs.fedoraproject.org/gitweb/?p=ibus-m17n.git;a=blob;f=ibus-m17n-xkb-...

[Peng, 2011-09-02 14:41:29]

On 2011/09/02 06:55:40, Daiki Ueno wrote:
> On 2011/09/02 06:52:18, Daiki Ueno wrote:
> > In Fedora, we have several options defined in default.xml.  However, the
> current
> > default.xml is not so efficient that those options can be maintained easily,
> > because every engine shall match with only one <engine> element.
> > 
> > This patch allows value override like fontconfig configuration files.
> 
> For example, this patch reduces the amount of default.xml.in.in in the
following
> patch:
> 
>
http://pkgs.fedoraproject.org/gitweb/?p=ibus-m17n.git;a=blob;f=ibus-m17n-xkb-...
> 
> to:
> 
>
http://pkgs.fedoraproject.org/gitweb/?p=ibus-m17n.git;a=blob;f=ibus-m17n-xkb-...

LGTM with some comments.

BTW, I am thinking if we could add a hide/disable option in the xml file, and
then we could use it to hide some keymaps.

[Peng, 2011-09-02 14:41:37]

http://codereview.appspot.com/4950059/diff/1/src/m17nutil.c
File src/m17nutil.c (right):

http://codereview.appspot.com/4950059/diff/1/src/m17nutil.c#newcode26
src/m17nutil.c:26: 
Please remove this empty line.

[Daiki Ueno, 2011-09-05 00:43:13]

On 2011/09/02 14:41:29, Peng wrote:

> BTW, I am thinking if we could add a hide/disable option in the xml file, and
> then we could use it to hide some keymaps.

Maybe we could reuse <rank> element with negative value for disabled keymaps. 
I'll prepare a CL.

[Daiki Ueno, 2011-09-05 01:09:04]

On 2011/09/05 00:43:13, Daiki Ueno wrote:
> On 2011/09/02 14:41:29, Peng wrote:
> 
> > BTW, I am thinking if we could add a hide/disable option in the xml file,
and
> > then we could use it to hide some keymaps.
> 
> Maybe we could reuse <rank> element with negative value for disabled keymaps. 
> I'll prepare a CL.

Oh, we have already the code:

https://github.com/ibus/ibus-m17n/blob/master/src/m17nutil.c#L195

Do you have any keymaps in mind to be disabled by default?