Summary:ASTERISK-01360: [patch] stdint.h nonexistent on FreeBSD
Reporter:Olle Johansson (oej)Labels:
Date Opened:2004-04-06 11:20:37Date Closed:2004-09-25 02:52:16
Versions:Frequency of
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