On Tue, 13 Feb 2007, Graeme Geldenhuys wrote:
> On 2/13/07, Michael Van Canneyt <[EMAIL PROTECTED]> wrote:
> > > Searching the internet it seems the XML preferred format for a empty
> > > element is a single tag with a '/' at the end. I guess we need to
> > > modify 'makeskel' then to output the correct/preferred format.
> >
> > No, that defeats the purpose of makeskel.
>
> Not sure what you mean by this? A single empty tag takes up less
> space that a open/close empty tag.
I know, but for someone editing the xml in a text editor (like me),
having the tags there means less typing. That was the original purpose.
I guess an extra option would be in order.
>
> >
> > The point is that the check for 'undocumented' in lazdoc checks for the
> > presence of a short/descr node. It should also check whether there is
> > actual text inside the node.
>
> I disagree. Changing the node to the following LazDoc shows correctly
> that it *isn't* documented. A single empty tag or a open/close tag on
> one line works. The XSLT file I posted earlier creates single "empty"
> tags and LazDoc is perfectly happy with that, saying that no
> documenation as been inserted.
All depends on the nodes :-)
>
> --------------------- BEGIN --------------------------
> <!-- enumeration type Visibility: default -->
> <element name="TCriteriaType">
> <short/>
> <descr></descr>
> <seealso></seealso>
> </element>
> --------------------- END --------------------------
>
>
> I think 'makeskel' still needs some cleanup - as well as LazDoc. Why
> does 'makeskel' create some nodes with a single tag (<sometag/>) and
> sometimes it creates nodes with a open/close "empty" tag? That's
> inconsistent.
Indeed. I'm open for patches. But the current way of having <tag></tag>
should also be preserved.
>
> Also regarding LazDoc. From what I could find on the internet, though
> not sure how official this is, but a whitespace characters (space,
> tab, Cr, Lf) does not coun't as data in a element.
It's a confused matter and one of the issues with XML...
Michael.
_________________________________________________________________
To unsubscribe: mail [EMAIL PROTECTED] with
"unsubscribe" as the Subject
archives at http://www.lazarus.freepascal.org/mailarchives
