[prev in list] [next in list] [prev in thread] [next in thread] 

List:       koffice-devel
Subject:    Re: Interesting QDomDocument::setContent variant
From:       Thomas Zander <zander () kde ! org>
Date:       2004-06-08 12:58:49
Message-ID: 200406081458.50526.zander () kde ! org
[Download RAW message or body]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Tuesday 08 June 2004 14:15, David Faure wrote:
> > Perhaps we can convince TT to add a method like:
> > QPtrList<QDomElement> QDomElement::children(const QString &);
>
> You can try, but I doubt you will succeed, since it's not in the standard
> DOM API.

Extending a class with more functionality will hardly make it non-DOM 
compatible..

> The QDomNode class docu even has sample code with a while loop that
> tests for isElement() correctly. If most KDE developers (me included) were
> too dumb to follow the documentation, that's more our fault than theirs :-)

IMO thats a bit naive; standard OO practices dictate that if an API dictates 
code-duplication the API needs to be changed or extended.

Then again; I will probably now be using the QDom API anyway; so I'm certainly 
not going to press this.

- -- 
Thomas Zander
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.3 (GNU/Linux)

iD8DBQFAxbgJCojCW6H2z/QRAt+kAKClNSI1KlCKs3GA9RnOn5eSqDY/aQCffxq/
h2YRYvIfjAuiAjiUpI16kAk=
=A9x6
-----END PGP SIGNATURE-----
_______________________________________________
koffice-devel mailing list
koffice-devel@mail.kde.org
https://mail.kde.org/mailman/listinfo/koffice-devel

[prev in list] [next in list] [prev in thread] [next in thread]