[PATCH] lib: Fix compilation error in C++ due to inline function

[PATCH] lib: Fix compilation error in C++ due to inline function

[Richard Palethorpe]

This adds explicit typecasts to an inline function to make it C++ compatible.
---
 lib/bluetooth.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/bluetooth.h b/lib/bluetooth.h
index 6ca64b6..ab24443 100644
--- a/lib/bluetooth.h
+++ b/lib/bluetooth.h
@@ -345,8 +345,8 @@ typedef struct {
 
 static inline void bswap_128(const void *src, void *dst)
 {
-	const uint8_t *s = src;
-	uint8_t *d = dst;
+	const uint8_t *s = (const uint8_t *)src;
+	uint8_t *d = (uint8_t *)dst;
 	int i;
 
 	for (i = 0; i < 16; i++)
-- 
2.3.4

Re: [PATCH] lib: Fix compilation error in C++ due to inline function

[Szymon Janc]

Hi Richard,

On Tuesday 24 of March 2015 14:52:24 Richard Palethorpe wrote:
> This adds explicit typecasts to an inline function to make it C++
> compatible. ---
>  lib/bluetooth.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/lib/bluetooth.h b/lib/bluetooth.h
> index 6ca64b6..ab24443 100644
> --- a/lib/bluetooth.h
> +++ b/lib/bluetooth.h
> @@ -345,8 +345,8 @@ typedef struct {
> 
>  static inline void bswap_128(const void *src, void *dst)
>  {
> -	const uint8_t *s = src;
> -	uint8_t *d = dst;
> +	const uint8_t *s = (const uint8_t *)src;
> +	uint8_t *d = (uint8_t *)dst;
>  	int i;

Just a nitpick: there should be space after cast
(see M5 in doc/coding-style.txt).

>  	for (i = 0; i < 16; i++)

Other than that patch looks good to me.

-- 
BR
Szymon Janc

Re: [PATCH] lib: Fix compilation error in C++ due to inline function

[Richard Palethorpe]

Hello Szymon,

Sorry, I haven't submitted a patch before, so should I add the spaces
then resubmit it as [PATCH V2] with it signed off by you?

On Fri, Mar 27, 2015 at 7:40 AM, Szymon Janc <szymon.janc@tieto.com> wrote:
> Hi Richard,
>
> On Tuesday 24 of March 2015 14:52:24 Richard Palethorpe wrote:
>> This adds explicit typecasts to an inline function to make it C++
>> compatible. ---
>>  lib/bluetooth.h | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/lib/bluetooth.h b/lib/bluetooth.h
>> index 6ca64b6..ab24443 100644
>> --- a/lib/bluetooth.h
>> +++ b/lib/bluetooth.h
>> @@ -345,8 +345,8 @@ typedef struct {
>>
>>  static inline void bswap_128(const void *src, void *dst)
>>  {
>> -     const uint8_t *s = src;
>> -     uint8_t *d = dst;
>> +     const uint8_t *s = (const uint8_t *)src;
>> +     uint8_t *d = (uint8_t *)dst;
>>       int i;
>
> Just a nitpick: there should be space after cast
> (see M5 in doc/coding-style.txt).
>
>>       for (i = 0; i < 16; i++)
>
> Other than that patch looks good to me.
>
> --
> BR
> Szymon Janc



-- 
Many thanks,

Richard Palethorpe

Software Developer

R&D Department

Re: [PATCH] lib: Fix compilation error in C++ due to inline function

[Szymon Janc]

On Friday 27 of March 2015 10:53:05 Richard Palethorpe wrote:
> Hello Szymon,
> 
> Sorry, I haven't submitted a patch before, so should I add the spaces
> then resubmit it as [PATCH V2] with it signed off by you?

Yes, please resend as [PATCH V2]. Since we don't use SOB in userspace code 
don't add any.

Also put sample compilation error in commit message (we usually do that for 
compilation fix patches).

And please don't top post.

> 
> On Fri, Mar 27, 2015 at 7:40 AM, Szymon Janc <szymon.janc@tieto.com> wrote:
> > Hi Richard,
> > 
> > On Tuesday 24 of March 2015 14:52:24 Richard Palethorpe wrote:
> >> This adds explicit typecasts to an inline function to make it C++
> >> compatible. ---
> >> 
> >>  lib/bluetooth.h | 4 ++--
> >>  1 file changed, 2 insertions(+), 2 deletions(-)
> >> 
> >> diff --git a/lib/bluetooth.h b/lib/bluetooth.h
> >> index 6ca64b6..ab24443 100644
> >> --- a/lib/bluetooth.h
> >> +++ b/lib/bluetooth.h
> >> @@ -345,8 +345,8 @@ typedef struct {
> >> 
> >>  static inline void bswap_128(const void *src, void *dst)
> >>  {
> >> 
> >> -     const uint8_t *s = src;
> >> -     uint8_t *d = dst;
> >> +     const uint8_t *s = (const uint8_t *)src;
> >> +     uint8_t *d = (uint8_t *)dst;
> >> 
> >>       int i;
> > 
> > Just a nitpick: there should be space after cast
> > (see M5 in doc/coding-style.txt).
> > 
> >>       for (i = 0; i < 16; i++)
> > 
> > Other than that patch looks good to me.
> > 
> > --
> > BR
> > Szymon Janc

-- 
BR
Szymon Janc