| Summary: | Missing inclusion of inttypes.h in dbus/dbus-sysdeps.h | ||
|---|---|---|---|
| Product: | dbus | Reporter: | Dagobert Michelsen <dam> |
| Component: | core | Assignee: | Havoc Pennington <hp> |
| Status: | RESOLVED FIXED | QA Contact: | John (J5) Palmieri <johnp> |
| Severity: | normal | ||
| Priority: | medium | ||
| Version: | 1.5 | ||
| Hardware: | SPARC | ||
| OS: | Solaris | ||
| Whiteboard: | |||
| i915 platform: | i915 features: | ||
| Attachments: | Add missing inclusion of inttypes.h | ||
|
Description
Dagobert Michelsen
2011-08-23 08:21:40 UTC
Created attachment 50499 [details] [review] Add missing inclusion of inttypes.h stdint.h and inttypes.h are both C99 headers. If one is not available, it's likely the other isn't either. Review of attachment 50499 [details] [review]: Patch looks sane. I'm guessing you have such a system where one C99 header is present but the other isn't. Yes, this is the case. The same construct is in dbus-hash.h and dbus-pipe.h, just dbus-sysdeps.h was missing the additional include. Thanks, fixed in git for 1.4.16, 1.5.8. (But, seriously? It's 2011 and systems without stdint.h, specified in C99 12 years ago, are still relevant? Please complain to the platform vendor.) (In reply to comment #5) > (But, seriously? It's 2011 and systems without stdint.h, specified in C99 12 > years ago, are still relevant? Please complain to the platform vendor.) Unfortunately, yes. Microsoft, for example, has said they have no plans to add C99 support to Visual Studio. Please complain to the vendor anyway. |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.