Summary: | ASTERISK-01360: [patch] stdint.h nonexistent on FreeBSD | ||
Reporter: | Olle Johansson (oej) | Labels: | |
Date Opened: | 2004-04-06 11:20:37 | Date Closed: | 2004-09-25 02:52:16 |
Priority: | Major | Regression? | No |
Status: | Closed/Complete | Components: | Core/General |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) md5-h.txt ( 1) patch-include_asterisk_frame_h ( 2) patch-include_asterisk_md5_h ( 3) patch-md5_c | |
Description: | ...seems like I may replace in with inttypes.h | ||
Comments: | By: Olle Johansson (oej) 2004-04-06 11:22:20 Don't know if this also applies to OpenBSD. Anyone that can check? By: James Golovich (jamesgolovich) 2004-04-06 12:17:02 D'oh. Someone assured me that stdint.h existed on freebsd, but apparantly they were wrong. Does inttypes.h contain uint32_t? By: Olle Johansson (oej) 2004-04-06 13:12:59 Complete version: ------------------- /* * This file is in the public domain. * $FreeBSD: src/sys/sys/inttypes.h,v 1.2 1999/08/28 00:51:47 peter Exp $ */ #ifndef _SYS_INTTYPES_H_ #define _SYS_INTTYPES_H_ #include <machine/ansi.h> typedef __int8_t int8_t; typedef __int16_t int16_t; typedef __int32_t int32_t; typedef __int64_t int64_t; typedef __uint8_t uint8_t; typedef __uint16_t uint16_t; typedef __uint32_t uint32_t; typedef __uint64_t uint64_t; typedef __intptr_t intptr_t; typedef __uintptr_t uintptr_t; #endif /* !_SYS_INTTYPES_H_ */ By: jcs (jcs) 2004-04-06 13:23:53 uploaded openbsd compatibility patch i also have openbsd patches for md5.c and include/asterisk/frame.h, do you want those here or in a new bug? By: Olle Johansson (oej) 2004-04-06 13:29:39 sys/types.h works for FreeBSD as well. Much better, propably. Add the patches here as well, as long as they're related to portability... By: jcs (jcs) 2004-04-06 13:53:43 added 2 patches By: Olle Johansson (oej) 2004-04-06 13:59:31 When adding patches, it's good habit to explain the purpose of the patches with a line or two so we know what they're supposed to do... :-) By: James Golovich (jamesgolovich) 2004-04-06 14:17:08 Since inttypes.h on linux include stdint.h we can just include that instead. I'll fix that up and close this out. None of the other patches should be necessary By: James Golovich (jamesgolovich) 2004-04-06 14:23:10 This is fixed in CVS head. Anything unrelated should be opened as a different bug |