Re: [xml] Re: Gnome-xml bugs?

Date view Thread view Subject view Author view

From: Daniel Veillard (Daniel.Veillard@w3.org)
Date: Mon Feb 08 1999 - 13:35:04 EST

Quoting Alexander Larsson (alla@lysator.liu.se):
> On 8 Feb, Daniel Veillard wrote:
> >
> >> It previously gave this xml code: <root prop\>, but after my comment you
> >> changed it to generate <root prop=""\>. This has been changed back in
> >> the latest cvs version. Is this a bug, or have you deliberatly changed
> >> it? My program Dia currently depends on this.
> >
> > Fixed in CVS, sorry for reintroducing this bug,
>
> Hmm, it seems the other bug is also left.

  Oops, yes ...

> Parsing <root prop=""\> gives the property a value of NULL when
> doing xmlNodeListGetString(). I think it should give strdup("")
> instead. (Because then you can separate xmlGetProp() return status for
> 'non-existing property' and 'property value is empty string')

  Rather than changing xmlNodeListGetString() I prefer changing
xmlGetProp to catch the case where the attribute exist but the associed
node list is NULL. xmlNodeListGetString is a bit more generic, and
it doesn't make sense to have it return "" when a NULL pointer is
given,

    I hope it's Ok, I commited that change,

 Daniel 

-- 
	    [Yes, I have moved back to France !]
Daniel.Veillard@w3.org | W3C  INRIA Rhone-Alpes  | Today's Bookmarks :
Tel : +33 476 615 257  | 655, avenue de l'Europe | Linux, WWW, rpmfind,
Fax : +33 476 615 207  | 38330 Montbonnot FRANCE | rpm2html, Kaffe,
http://www.w3.org/People/W3Cpeople.html#Veillard | badminton, and Amaya.
----
Message from the list xml@rufus.w3.org
Archived at : http://rufus.w3.org/veillard/XML/messages
to unsubscribe: echo "unsubscribe xml" | mail  majordomo@rufus.w3.org

Date view Thread view Subject view Author view

This archive was generated by hypermail 2b29 : Wed Aug 02 2000 - 12:29:25 EDT