P , 2012-06-04 09:35 +0100, Chris Morley rakstīja:
> On 03/06/2012 17:58, Geoff Hutchison wrote:
> >> Right, it came in C++ from C and should be in most compilers now. I also
> >> think that this is the best option. I'll write a patch for it. Only I'm
> >> reading that inttypes.h is not available for VS2003-2008, do we need a
> >> workaround for them? If so, maybe it is better to just define uint32_t
> >> myself.
> >
> > I think it's safe to say that VS2003-VS2008 are compiling for 32-bit 
> > Windows. Maybe Chris can comment if 64-bit Windows uses 64-bit integers.
> 
> Noel has a 64 bit Windows build, which is on the dashboard and may or 
> may not appear in the release. 64 bit builds with Visual Studio use a 32 
> bit int.

I have committed r4854 with checks for VS version. For older versions
define uint32_t, for all other include inttypes.h, like this:

#if defined(_MSC_VER) && _MSC_VER < 1600
  // Assuming 32bit integer
  typedef unsigned uint32_t;
#else
  #include <inttypes.h>
#endif

I haven't seen Windows builds on dashboard since 2012.05.26, hopefully
they will come back and I'll see if something breaks.


Reinis


------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
OpenBabel-Devel mailing list
OpenBabel-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openbabel-devel

Reply via email to