diff options
author | Yves Orton <demerphq@gmail.com> | 2020-02-08 16:02:49 +0100 |
---|---|---|
committer | Yves Orton <demerphq@gmail.com> | 2020-02-08 16:03:31 +0100 |
commit | 4b16e2bb0cafabffef653e5dad060dbe68bddcf9 (patch) | |
tree | 17534babee4a032ed3af7e665dbd3cadb8f3708d /win32/win32.h | |
parent | 9e831ddf560754471ff3a651fa851b7941db927a (diff) | |
download | perl-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 'win32/win32.h')
-rw-r--r-- | win32/win32.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/win32/win32.h b/win32/win32.h index a91776cf0e..1133c60606 100644 --- a/win32/win32.h +++ b/win32/win32.h @@ -254,9 +254,9 @@ struct utsname { /* VC uses non-standard way to determine the size and alignment if bit-fields */ /* MinGW will compile with -mms-bitfields, so should use the same types */ -#define PERL_BITFIELD8 unsigned char -#define PERL_BITFIELD16 unsigned short int -#define PERL_BITFIELD32 unsigned long int +#define PERL_BITFIELD8 U8 +#define PERL_BITFIELD16 U16 +#define PERL_BITFIELD32 U32 #ifdef _MSC_VER /* Microsoft Visual C++ */ |