rpms/advancecomp/devel advancecomp-64bit.patch, NONE, 1.1 advancecomp.spec, 1.8, 1.9

Matthias Saou thias at spam.spam.spam.spam.spam.spam.spam.egg.and.spam.freshrpms.net
Fri May 27 08:25:36 UTC 2005


Ralf Corsepius wrote :

> On Thu, 2005-05-26 at 11:12 -0400, Jeremy Katz wrote:
> 
> > --- advancecomp-1.14/7z/Portable.h.64bit	2005-05-26 11:02:30.000000000 -0400
> > +++ advancecomp-1.14/7z/Portable.h	2005-05-26 10:50:22.000000000 -0400
> > @@ -16,7 +16,11 @@
> >  typedef UINT16 WORD;
> >  typedef UINT32 DWORD;
> >  
> > -typedef unsigned UINT_PTR;
> > +#if defined(__x86_64__) || defined(__s390x__) || defined(__ppc64__) || defined(__ia64__) || defined(__sparc64__) || defined(__alpha__)
> > +typedef unsigned long UINT_PTR;
> > +#else
> > +typedef unsigned int UINT_PTR;
> > +#endif
> A much better approach would be to use c99/stdint.h types, instead.
> 
> i.e.
> 
> #include <stdint.h>
> 
> typedef uint16_t WORD;
> typedef uint32_t DWORD;
> typedef uintptr_t UINT_PTR;

I'll try that approach, as it seems cleaner than iterating 64bit archs,
even if the end result is the same.

Matthias

-- 
Clean custom Red Hat Linux rpm packages : http://freshrpms.net/
Fedora Core release 3 (Heidelberg) - Linux kernel 2.6.11-1.27_FC3
Load : 0.29 0.38 0.37




More information about the fedora-extras-commits mailing list