Re: [Bluez-devel] Proposed DTD

Marcel Holtmann
2006-10-24T10:17:34Z

Hi Denis,

> > > Yes, the encoding is only relevant for the <text[8,16,32]> fields. The
> > > rest of the attributes should not have it (this is hard to express in a
> > > DTD.)
> >
> > then don't express it at all. Leave it out. An XML file can be valid
> > based on its DTD, but still be rejected when stuff is not passing the
> > semantic requirements.

I prefer to leave it out now and then add it later if really needed. It
is simpler to add something than to remove it.

> I'd rather leave it, encoding attribute is optional anyway, and as you said
> it'll only happen for HID profile. Might as well treat the entire thing as a
> data blob.
>
> > If I can get an XML parser that only deals with this specific DTD and is
> > not using expat at all, I am willing to include it right away and make
> > sdptool use it.
>
> Cool. I will be unable to work on this for a few weeks, so if nobody steps up
> till then I'll see what I can do. In the interest of getting something
> working quickly and generating feedback, would you be willing to introduce a
> dependency on expat for hcid? The XML infrastructure really belongs in the
> new dbus API anyway.

Having a temporary expat dependency is not an option am willing to go
for. However removing the code from sdptool and only put some XML
templates in a shared place seems like a good option.

Regards

Marcel