Opened 6 years ago

Closed 5 years ago

Last modified 5 years ago

#15 closed defect (fixed)

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 (2)

openobex-03-CC-only.diff​ (459 bytes) - added by jerryyu 6 years ago.
patch for this bug
openobex-uint8.patch​ (692 bytes) - added by zany 6 years ago.
replace u_int8_t with uint8_t

Download all attachments as: .zip

Change History (5)

Changed 6 years ago by jerryyu

patch for this bug

comment:1 Changed 6 years 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 6 years ago by zany

replace u_int8_t with uint8_t

comment:2 Changed 6 years 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.

comment:3 Changed 5 years ago by zany

  • Resolution set to fixed
  • Status changed from assigned to closed

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

Note: See TracTickets for help on using tickets.