[xml] Possible 64-bit issues in xml 2.9.2

2015-11-10 Thread Bruce Dawson
nd that is not the case with VC++. I think that the correct type would be ptrdiff_t, or size_t, or intptr_t, or uintptr_t. I've attached the full set of warnings in case that is of any assistance. Even though some of these warnings do not indicate actual bugs it would still be best to use th

Re: [xml] Possible 64-bit issues in xml 2.9.2

2015-11-10 Thread Bruce Dawson
tems). And horsepower on these boxes is so high that the translation is > not hurting. I have sites that literally exchange 1.5 million - 2.0 > million XML files per day (most in 10 hours) on a weenie little IBM using > libxml2 to parse and create XML. > > My point on this is you may

Re: [xml] Possible 64-bit issues in xml 2.9.2

2015-11-10 Thread Bruce Dawson
use for those, I just > wonder what kind of old arches will break if they don't have it. > > Can you come up with a patch which just change the casts and possibly the > target > types ? > > Daniel > > -- > Daniel Veillard | Open Source and Standards, Red Hat > veill...@redhat.com | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ > http://veillard.com/ | virtualization library http://libvirt.org/ > -- Bruce Dawson ___ xml mailing list, project page http://xmlsoft.org/ xml@gnome.org https://mail.gnome.org/mailman/listinfo/xml

[xml] win32\VC10\config.h and VS 2015

2016-03-07 Thread Bruce Dawson
/ Cannot define this in VS 2015 and above! #define snprintf _snprintf +#endif #if _MSC_VER < 1500 #define vsnprintf(b,c,f,a) _vsnprintf(b,c,f,a) #endif It would be great to get that same patch in libxml2, and other developers will need it if they use VC10\config.h. -- B

[xml] VC10\config.h and include\win32config.h

2016-03-07 Thread Bruce Dawson
lines: #define SEND_ARG2_CAST #define GETHOSTBYNAME_ARG_CAST Which version should we prefer, and can we get the other version deleted from libxml2 to avoid confusion? -- Bruce Dawson ___ xml mailing list, project page http://xmlsoft.org/ xml@gnome.org

Re: [xml] VC10\config.h and include\win32config.h

2016-03-11 Thread Bruce Dawson
y we would have multiple > version rotting in multiple directories, unappealing indeed ! > > > In any case, I think win32config.h should be updated; see below for my > > responses to Bruce's comments about the differences between the configs. > > Sure, if people who are u

Re: [xml] win32\VC10\config.h and VS 2015

2016-03-11 Thread Bruce Dawson
:46, Bruce Dawson wrote: > >> The config.h file in win32\VC10 is incompatible with VS 2015 because VS >> 2015 >> provides an implementation of snprintf and prohibits using the >> preprocessor to >> define it. Therefor an ifdef check is needed around that definition

[xml] VS 2015 code-gen bug triggered by libxml

2016-04-12 Thread Bruce Dawson
similar fix upstream and perhaps a test of xmlUTF8Strsize()'s behavior might be a good idea. The bug does not occur when optimizing for speed. FWIW. -- Bruce Dawson ___ xml mailing list, project page http://xmlsoft.org/ xml@gnome.org https://mail.gnom

[xml] Minor improvement possible

2017-01-05 Thread Bruce Dawson via xml
it should be applicable to libxml2) is attached. -- Bruce Dawson commit eafa8bbee03e8ebba366be1fb8d5b1596a989a0a Author: Bruce Dawson Date: Thu Jan 5 11:25:58 2017 -0800 Add const in five places to move 1 KiB to .rdata xmlUnicodeBlocks is logically const but was not marked as

Re: [xml] Minor improvement possible

2017-01-10 Thread Bruce Dawson via xml
best practice even for single-processing. It's not a huge advantage, but it is strictly better than having logically-const globals in the read/write data segment. On Thu, Jan 5, 2017 at 3:00 PM, Bruce Dawson wrote: > The constant array xmlUnicodeBlocks is not marked as 'const'