#8357 closed patch
Possible fix for inttypes.h problem
| Reported by: | eriktorbjorn | Owned by: | fingolfin |
|---|---|---|---|
| Priority: | normal | Component: | --Other-- |
| Version: | Keywords: | ||
| Cc: | Game: |
Description
The discussion on the mailing list seems to have fallen silent, so here's one possible way of dealing with the inttypes.h problem. As you can see, it simply adds a check for inttypes.h to the configure script and then uses the result from that to determine if it should include inttypes.h or if it should do its own typedeffing.
Mind you, the configure script already assumes that inttypes.h is present when it checks for libmpeg2, but I still thought it was more elegant to have a separate test for it.
Ticket imported from: #1040501. Ticket imported from: patches/462.
Attachments (2)
Change History (6)
by , 20 years ago
| Attachment: | inttypes.diff added |
|---|
comment:1 by , 20 years ago
Hm, why use HAVE_INTTYPES at all? Why not just always use those typedefs?
by , 20 years ago
| Attachment: | inttypes2.diff added |
|---|
Alternative patch against an October 16 CVS snapshot
comment:2 by , 20 years ago
As long as we only use inttypes.h to keep mpeg2.h happy, I guess HAVE_INTTYPES isn't needed, and this option was one of the reasons I didn't commit the patch. I figured whichever way I went someone would ask "but why didn't you...?", but I was too lazy then to do both. :-)
I've attached an alternative patch now, though.
comment:3 by , 20 years ago
| Owner: | set to |
|---|---|
| Status: | new → closed |
comment:4 by , 6 years ago
| Component: | → --Other-- |
|---|
Patch against an October 5 CVS snapshot