Re: [PATCH BlueZ 05/26] attrib: Get rid of guint* types

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Anderson,

On Thu, May 9, 2013 at 7:45 AM, Anderson Lizardo
<anderson.lizardo@xxxxxxxxxxxxx> wrote:
> Hi Lucas,
>
> On Thu, May 9, 2013 at 12:16 AM, Lucas De Marchi
> <lucas.demarchi@xxxxxxxxxxxxxx> wrote:
>> guint -> unsigned int
>> guint8 -> uint8_t
>> guint16 -> uint16_t
>> guint32 -> uint32_t
>> guint64 -> uint64_t
>>
>> Add "#include <inttypes.h>" where appropriate and prefer this one over
>> stdint.h.
>> ---
>>  attrib/att.c         |   2 +-
>>  attrib/gatt.c        | 124 ++++++++++++++++++++++++++-------------------------
>>  attrib/gatt.h        |  33 +++++++-------
>>  attrib/gattrib.c     |  51 ++++++++++-----------
>>  attrib/gattrib.h     |  21 +++++----
>>  attrib/gatttool.c    |  19 ++++----
>>  attrib/interactive.c |  35 ++++++++-------
>>  7 files changed, 148 insertions(+), 137 deletions(-)
>>
>> diff --git a/attrib/att.c b/attrib/att.c
>> index d2b78bd..e196fa3 100644
>> --- a/attrib/att.c
>> +++ b/attrib/att.c
>> @@ -27,7 +27,7 @@
>>  #endif
>>
>>  #include <errno.h>
>> -#include <stdint.h>
>> +#include <inttypes.h>
>>  #include <stdlib.h>
>
> Any reason to include inttypes.h instead of just stdint.h? Are we
> using any of the PRI* format conversion macros?

The reason is to be able to use PRI formats without having to change
the include later.


Lucas De Marchi
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux