On Wed, 19 Sep 2012, Roumen Petrov wrote:

This is because HAVE_CONFIG_H is not #defined, which is odd, because configure.js goes to the trouble of creating a config.h file.

No please do not introduce again HAVE_CONFIG_H .
Please fix the test as in https://bugzilla.gnome.org/show_bug.cgi?id=681228

How should this be fixed? MSVC7.1 has "_vsnprintf", not "vsnprintf". How do you want to redirect "vsnprintf" in the code to "_vsnprintf" at link time?

(Note: The above names are actually "_vsnprintf" and "__vsnprintf" as far as the linker is concerned, thanks to Microsoft convention.)

I don't think getting rid of config.h for Win32 builds is a good idea, because then you no longer have a common place to put in workarounds for the Windows code environment. The leading underscore on vsnprintf() is just one example of many.


--Daniel


--
Daniel Richard G. || [email protected] || Software Developer
Teragram Linguistic Technologies (a division of SAS)
http://www.teragram.com/

_______________________________________________
xml mailing list, project page  http://xmlsoft.org/
[email protected]
https://mail.gnome.org/mailman/listinfo/xml

Reply via email to