![]() |
|
Allegro 4.9.20 released |
Mark Oates
Member #1,146
March 2001
![]() |
Trent Gamblin said: Wow that's weird. Yea. I just overwrote the previous files like I would with any other of your new builds. Peculiar. -- |
Trent Gamblin
Member #261
April 2000
![]() |
Ok, here's a patch that should fix it. I'll see if it's been done in svn and apply it there if not.
|
Matthew Leverton
Supreme Loser
January 1999
![]() |
Isn't stdint.h included elsewhere? |
Trent Gamblin
Member #261
April 2000
![]() |
Apparently not, because it doesn't work. If you have a better solution, by all means... I avoid MSVC when I can.
|
Matthew Leverton
Supreme Loser
January 1999
![]() |
I think all he needs to do is this in alplatf.h: #define ALLEGRO_HAVE_STDINT_H
Since you created it with MSVC 9, it is left commented out. (I guess ... it's in 10.) It's not really a bug... the headers aren't guaranteed to be compatible across different compilers or platforms. Edit: Or maybe MSVC 9 has stdint.h too? I don't remember for sure, but it seems like it shouldn't be a new problem to 4.9.20. I'm pretty sure I've compiled for 9 and 10 without problems. |
Trent Gamblin
Member #261
April 2000
![]() |
Yeah, that sounds about right.
|
Matthew Leverton
Supreme Loser
January 1999
![]() |
Well, base.h includes alconfig.h which includes astdint.h which conditionally includes stdint.h. For MSVC 10, ALLEGRO_HAVE_INTTYPES_H should not be defined and ALLEGRO_HAVE_STDINT_H should be (in alplatf.h). Edit: Yeah, your file is missing that define. If you change your alplatf.h in the binary package to look like: #if _MSC_VER >= 1600 #define ALLEGRO_HAVE_STDINT_H #endif
it should work for 9 and 10. Although, I thought MSVC 9 had stdint (but was missing int64/uint64)... Edit 2: What's confusing is why almsvc.h has: #if _MSC_VER < 1600 #define int64_t signed __int64 #define uint64_t unsigned __int64 #endif I know I added the #if _MSC_VER < 1600 check, but it seems if those two #defines were with the rest of the guesswork in astdint.h, it wouldn't even be necessary. Anyway, the proper thing for Mark to do is add ALLEGRO_HAVE_STDINT_H to alplatf.h. |
Mark Oates
Member #1,146
March 2001
![]() |
Matthew Leverton said: #define ALLEGRO_HAVE_STDINT_H K, that did it. Thanks, guys. I was also getting GLINT errors when including allegro_opengl.h, also related to the int not being defined. -- |
Michał Cichoń
Member #11,736
March 2010
|
MSVC 2010 in fact do have stdint.h, all previous version does not. That's main reason why I provided different headers sets in my builds. "God starts from scratch too" |
SiegeLord
Member #7,827
October 2006
![]() |
Michał Cichoń said: I just want to bump one bug. Currently D3D driver crashes while closing display. Elias said something about revisiting destruction order. I'm don't know internal Allegro design, so I don't think I'm good man for the job. You can find more info there.
This should be fixed in the SVN now... although upgrading to SVN might be troublesome since we just... broke every single A5 program out there "For in much wisdom is much grief: and he that increases knowledge increases sorrow."-Ecclesiastes 1:18 |
Mark Oates
Member #1,146
March 2001
![]() |
-- |
Michał Cichoń
Member #11,736
March 2010
|
Nice, I will check it out. : ) Wow. Nice crash. Allegro now crashes on destroying dummy display in al_uninstall_system. I tried both: D3D and OpenGL, same result. "God starts from scratch too" |
Trent Gamblin
Member #261
April 2000
![]() |
Can you try that again? I put the free too late.
|
Michał Cichoń
Member #11,736
March 2010
|
Now it is OK. Thanks. "God starts from scratch too" |
|
|