"PeterPan" <<zenothing[at]hotmail.com>> wrote:
> If you request "not print '/>' for all non-open tags", it result to no
> "/>" at all.
Yes. "/>" means a tag without content, but your patch also uses it for
tags with content and a matching end tag, which is confusing. If you
want to show open tags in the sprintf output, use some other kind of
indication.
Btw, I reworked the _sprintf function a bit to correct the recursion
problem, since it both was too recursive and produced too much output
to be suitable for %O. You might want to rebase your work on the cvs
fresh version (see http://pike.ida.liu.se/development/cvs/).
|