summaryrefslogtreecommitdiff
path: root/perl.h
diff options
context:
space:
mode:
authorYves Orton <demerphq@gmail.com>2020-02-08 16:02:49 +0100
committerYves Orton <demerphq@gmail.com>2020-02-08 16:03:31 +0100
commit4b16e2bb0cafabffef653e5dad060dbe68bddcf9 (patch)
tree17534babee4a032ed3af7e665dbd3cadb8f3708d /perl.h
parent9e831ddf560754471ff3a651fa851b7941db927a (diff)
downloadperl-4b16e2bb0cafabffef653e5dad060dbe68bddcf9.tar.gz
perl.h: define PERL_BITFIELD8/16/32 as U8/U16/U32
After consultation with xenu we decided that defining both sets exactly the same as U8/U16/U32 makes the most sense. These defines were added in 654eccd594bfe8deab367b0f4cdda726a7796ff3 and the discussion for why is here: https://www.nntp.perl.org/group/perl.perl5.porters/2008/01/msg133754.html Short summary: Activestate added these defines so that XS code compiled with gcc on windows could be used with a perl compiled with VC on windows, and vice-versa. Read the thread for more details. This patch does NOT remove the duplicate definition in win32/win32.h, I don't feel comfortable doing that without being able to check the build, so I will leave it to someone who has VC to test.
Diffstat (limited to 'perl.h')
-rw-r--r--perl.h6
1 files changed, 3 insertions, 3 deletions
diff --git a/perl.h b/perl.h
index 9198ee03ef..65009e117d 100644
--- a/perl.h
+++ b/perl.h
@@ -3827,13 +3827,13 @@ typedef struct crypt_data { /* straight from /usr/include/crypt.h */
/* macros to define bit-fields in structs. */
#ifndef PERL_BITFIELD8
-# define PERL_BITFIELD8 unsigned char
+# define PERL_BITFIELD8 U8
#endif
#ifndef PERL_BITFIELD16
-# define PERL_BITFIELD16 unsigned short int
+# define PERL_BITFIELD16 U16
#endif
#ifndef PERL_BITFIELD32
-# define PERL_BITFIELD32 unsigned long int
+# define PERL_BITFIELD32 U32
#endif
#include "sv.h"