On Wed, Aug 16, 2006 at 05:00:16PM +0200, Roland Schwingel wrote:
> Here comes my revised/extended patch.

  okay, got it :-)

> What is the state now:
> In the case that a path cannot be accessed on disk asuming the path to be 
> in
> utf-8 on windows, it is also tried with native encoding now as fallback. 
> That should
> fix the first part.

[...]
> 
> When doing the patch I found 2 static functions in xmlIO.c doing quite the 
> same thing.
> xmlSysIDExists() and xmlNoNetExists(). In favour of simplicity I decided 
> to discard xmlSysIDExists().

  okay xmlNoNetExists was only compiled in if catalog where activated, 
but since xmlSysIDExists was always compiled in and the final version always
compile them, it's fine.

> So I hope this resolves all pending issues. Feel free to reply in case of 
> any problems.
[..]
> PS: Now with patch attached... Sorry...

  Just one last touch. It seems you open the mscrt DLL in 2 different place
in the resulting code. Can't this be shared as a single routine, guarded 
with a single static variable. This is relatively complex code, and I would
prefer a single instance of it, is that possible ?

Daniel

-- 
Red Hat Virtualization group http://redhat.com/virtualization/
Daniel Veillard      | virtualization library  http://libvirt.org/
[EMAIL PROTECTED]  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine  http://rpmfind.net/
_______________________________________________
xml mailing list, project page  http://xmlsoft.org/
[email protected]
http://mail.gnome.org/mailman/listinfo/xml

Reply via email to