qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] audio/mixeng_template.h: fix inline declaration
@ 2013-08-06 18:45 Alex Bligh
  2013-10-22  9:26 ` Alex Bligh
  2013-10-22  9:47 ` [Qemu-devel] " Peter Maydell
  0 siblings, 2 replies; 4+ messages in thread
From: Alex Bligh @ 2013-08-06 18:45 UTC (permalink / raw)
  To: qemu-devel; +Cc: Alex Bligh

Fix error: ‘inline’ is not at beginning of declaration [-Werror=old-style-declaration]

Signed-off-by: Alex Bligh <alex@alex.org.uk>
---
 audio/mixeng_template.h |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/audio/mixeng_template.h b/audio/mixeng_template.h
index 30849a6..77cc89b 100644
--- a/audio/mixeng_template.h
+++ b/audio/mixeng_template.h
@@ -35,7 +35,7 @@
 #define IN_T glue (glue (ITYPE, BSIZE), _t)
 
 #ifdef FLOAT_MIXENG
-static mixeng_real inline glue (conv_, ET) (IN_T v)
+static inline mixeng_real glue (conv_, ET) (IN_T v)
 {
     IN_T nv = ENDIAN_CONVERT (v);
 
@@ -54,7 +54,7 @@ static mixeng_real inline glue (conv_, ET) (IN_T v)
 #endif
 }
 
-static IN_T inline glue (clip_, ET) (mixeng_real v)
+static inline IN_T glue (clip_, ET) (mixeng_real v)
 {
     if (v >= 0.5) {
         return IN_MAX;
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [Qemu-devel] [PATCH] audio/mixeng_template.h: fix inline declaration
  2013-08-06 18:45 [Qemu-devel] [PATCH] audio/mixeng_template.h: fix inline declaration Alex Bligh
@ 2013-10-22  9:26 ` Alex Bligh
  2013-10-26  9:10   ` [Qemu-devel] [Qemu-trivial] " Michael Tokarev
  2013-10-22  9:47 ` [Qemu-devel] " Peter Maydell
  1 sibling, 1 reply; 4+ messages in thread
From: Alex Bligh @ 2013-10-22  9:26 UTC (permalink / raw)
  To: qemu-devel, qemu-trivial; +Cc: Alex Bligh

ping

I think this is should go to qemu-trivial

--On 6 August 2013 19:45:13 +0100 Alex Bligh <alex@alex.org.uk> wrote:

> Fix error: ‘inline’ is not at beginning of declaration
> [-Werror=old-style-declaration]
>
> Signed-off-by: Alex Bligh <alex@alex.org.uk>
> ---
>  audio/mixeng_template.h |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/audio/mixeng_template.h b/audio/mixeng_template.h
> index 30849a6..77cc89b 100644
> --- a/audio/mixeng_template.h
> +++ b/audio/mixeng_template.h
> @@ -35,7 +35,7 @@
>  #define IN_T glue (glue (ITYPE, BSIZE), _t)
>
>  #ifdef FLOAT_MIXENG
> -static mixeng_real inline glue (conv_, ET) (IN_T v)
> +static inline mixeng_real glue (conv_, ET) (IN_T v)
>  {
>      IN_T nv = ENDIAN_CONVERT (v);
>
> @@ -54,7 +54,7 @@ static mixeng_real inline glue (conv_, ET) (IN_T v)
>  #endif
>  }
>
> -static IN_T inline glue (clip_, ET) (mixeng_real v)
> +static inline IN_T glue (clip_, ET) (mixeng_real v)
>  {
>      if (v >= 0.5) {
>          return IN_MAX;
> --
> 1.7.9.5
>
>



-- 
Alex Bligh

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Qemu-devel] [PATCH] audio/mixeng_template.h: fix inline declaration
  2013-08-06 18:45 [Qemu-devel] [PATCH] audio/mixeng_template.h: fix inline declaration Alex Bligh
  2013-10-22  9:26 ` Alex Bligh
@ 2013-10-22  9:47 ` Peter Maydell
  1 sibling, 0 replies; 4+ messages in thread
From: Peter Maydell @ 2013-10-22  9:47 UTC (permalink / raw)
  To: Alex Bligh; +Cc: QEMU Developers

On 6 August 2013 19:45, Alex Bligh <alex@alex.org.uk> wrote:
> Fix error: ‘inline’ is not at beginning of declaration [-Werror=old-style-declaration]
>
> Signed-off-by: Alex Bligh <alex@alex.org.uk>

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

-- PMM

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Qemu-devel] [Qemu-trivial] [PATCH] audio/mixeng_template.h: fix inline declaration
  2013-10-22  9:26 ` Alex Bligh
@ 2013-10-26  9:10   ` Michael Tokarev
  0 siblings, 0 replies; 4+ messages in thread
From: Michael Tokarev @ 2013-10-26  9:10 UTC (permalink / raw)
  To: Alex Bligh; +Cc: qemu-trivial, qemu-devel

22.10.2013 13:26, Alex Bligh пишет:
> ping
>
> I think this is should go to qemu-trivial
>
> --On 6 August 2013 19:45:13 +0100 Alex Bligh <alex@alex.org.uk> wrote:
>
>> Fix error: ‘inline’ is not at beginning of declaration
>> [-Werror=old-style-declaration]

Thanks, applied to the trivial patches queue.

/mjt

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2013-10-26  9:10 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-08-06 18:45 [Qemu-devel] [PATCH] audio/mixeng_template.h: fix inline declaration Alex Bligh
2013-10-22  9:26 ` Alex Bligh
2013-10-26  9:10   ` [Qemu-devel] [Qemu-trivial] " Michael Tokarev
2013-10-22  9:47 ` [Qemu-devel] " Peter Maydell

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).