Simetrical schreef:
On Fri, May 23, 2008 at 5:36 PM, DanTMan dan_the_man@telus.net wrote:
Heh... we could even be using things like Xml::span which __callStatic wraps to Xml::element( 'span', ... );
Not if that's like the current __get and __set, which I assume it will be. Those are broken beyond belief: if one __get is nested within another, to any depth, it *silently* returns null. Purportedly to avoid infinite loops, but it's a spectacularly stupid way to do so. Extremely easy to cause bugs that will be a big pain to track down, unless you do almost nothing in the relevant functions or anything called by them.
That's the point, we *do* do almost nothing in Xml::* functions. They're either wrappers to other Xml::* functions (in which case all we have to do is watch out that we use real functions inside Xml::* functions, not __callStatic-powered fake ones) or one- or two-line returns.
Roan Kattouw (Catrope)