Ticket #15 (closed defect: fixed)

Opened 17 months ago

Last modified 4 months ago

uint8_t instead of u_int8_t is defined in Solaris

Reported by: jerryyu Owned by: zany
Priority: major Milestone:
Component: openobex-lib Version:
Keywords: Cc:

Description

At first if it's SUN OS will be checked. If yes, a macro will be defined "#define u_int8_t uint8_t.

Attachments

openobex-03-CC-only.diff (459 bytes) - added by jerryyu 17 months ago.
patch for this bug
openobex-uint8.patch (0.7 KB) - added by zany 17 months ago.
replace u_int8_t with uint8_t

Change History

Changed 17 months ago by jerryyu

patch for this bug

Changed 17 months ago by zany

  • owner changed from somebody to zany
  • status changed from new to assigned

This is not the right fix. u_int8_t should not be used as it is a linux kernel type. We should #include <inttypes.h> and use the C99 uint8_t.

Changed 17 months ago by zany

replace u_int8_t with uint8_t

Changed 17 months ago by jerryyu

It's very nice that u_int8_t was replacement with C99 uint8_t so that it can be compiled successfully for different platforms which support C99.

Changed 7 months ago by zany

  • status changed from assigned to closed
  • resolution set to fixed

Finally applied (with the win32 patch set). Thanks for spotting this Jerry!

Note: See TracTickets for help on using tickets.