Re: [xml] patch for adding "shell" scripting



Daniel Veillard <veillard redhat com> writes:

  I like the idea, but please rewitre the patch for portability.

Harumph.


  No, it's a portability nightmare...

I'm not very interested in portability  /8->

But I've done most of what you ask. See the next bit for some
questions:


  It would be good to make sure all string allocated or copied
are cleaned up with xmlFree() on exit too.

The script string created by the shellx argument processing is
allocated via malloc. Should I allocate that using xml memory
handlers? I'm a bit confused about what to use... free() is used in
the shell argument processor for example, not xmlFree().


  So good idea, but the code need some cleanup :-)

If I could spell a raspberry being blown I would.

Here's the patch:


Attachment: patch-xmllint-xmlshell
Description: Text Data



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]