Ruan Strydom ruan at
Tue Mar 23 06:10:17 UTC 2010

Hi it is in the repo, dtd is broken though, I tested it on html 4 loose.dtd 
and there is a number of tokens which is handled incorrectly (or not handled 
at all).  I am going through the specification on dtd, but still missing a 
couple of constructs. There is tons of problems and I tried to mark as many of 
them with fixme and todo's. I will add the problem dtd portions to the unit 

Besides not knowing how to make all the tags a tree in the parser, I though 
leaving them in a list will make it general enough for all dtd/tag based 
languages, but I suppose a flag could be used for parser behavior or if one 
exists the others can easily be derived from it.

Thanks for the help.

On Thursday 18 March 2010 23:06:26 Jonathan Schmidt-Dominé wrote:
> Hi Ruan!
> > 1) The parser only builds up an sequential array of elements (ast nodes),
> > except for doctype where the definitions is contained inside the doctype
> > ast node (in a dtd file it a an array again, the definitions is not
> > within a doctype element). Does this sound correct?
> > (ie <tag> <othertag/> </tag> <tag/> will be 4 ast nodes in the array not
> > three and one with a child)
> Niko is right, it should be a tree.
> > 2) I have 2 first/first and 1 first/last  conflict, but it appears to
> > work fine. The PG-QT web page says that can be ignored in some cases?
> Please tell me if the formulation is somehow complicated, is there anything
> not understandable, I would like to fix it? ;)
> > 3) Can I commit it so long and can someone have a look at the grammar. I
> > do not want to carry on if it is broken, since a lot of code (builders as
> > I understand it) will depend on it.
> Please commit it, I will have a look at it.
> Jonathan
> ------------------------
> Automatisch eingefügte Signatur:
> Es lebe die Freiheit!
> Stoppt den Gebrauch proprietärer Software!
> Operating System: GNU/Linux
> Kernel: Linux
> Distribution: openSuSE 11.2
> Qt: 4.6.3
> KDE: 4.4.67 (KDE 4.4.67 (KDE 4.5 >= 20100310)) "release 3"
> KMail: 1.13.1

More information about the KDevelop-devel mailing list