[PATCH] hw/i386/vmport: Drop superfluous parenthesis around function typedef

[PATCH] hw/i386/vmport: Drop superfluous parenthesis around function typedef

[Philippe Mathieu-Daudé]

Drop superfluous parenthesis around VMPortReadFunc typedef
(added in d67f679d99, missed to remove when moved in e595112985).

Suggested-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
 hw/i386/vmport.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hw/i386/vmport.h b/hw/i386/vmport.h
index 47eda7a22b..a6624e3045 100644
--- a/hw/i386/vmport.h
+++ b/hw/i386/vmport.h
@@ -27,7 +27,7 @@
 
 #define TYPE_VMPORT "vmport"
 
-typedef uint32_t (VMPortReadFunc)(void *opaque, uint32_t address);
+typedef uint32_t VMPortReadFunc(void *opaque, uint32_t address);
 
 void vmport_register(unsigned char command, VMPortReadFunc *func, void *opaque);
 
-- 
2.21.3

Re: [PATCH] hw/i386/vmport: Drop superfluous parenthesis around function typedef

[Philippe Mathieu-Daudé]

On 5/5/20 4:28 PM, Philippe Mathieu-Daudé wrote:
> Drop superfluous parenthesis around VMPortReadFunc typedef
> (added in d67f679d99, missed to remove when moved in e595112985).
> 
> Suggested-by: Richard Henderson <richard.henderson@linaro.org>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
>  hw/i386/vmport.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/hw/i386/vmport.h b/hw/i386/vmport.h
> index 47eda7a22b..a6624e3045 100644
> --- a/hw/i386/vmport.h
> +++ b/hw/i386/vmport.h
> @@ -27,7 +27,7 @@
>  
>  #define TYPE_VMPORT "vmport"
>  
> -typedef uint32_t (VMPortReadFunc)(void *opaque, uint32_t address);
> +typedef uint32_t VMPortReadFunc(void *opaque, uint32_t address);
>  
>  void vmport_register(unsigned char command, VMPortReadFunc *func, void *opaque);
>  
> 

ping?

Re: [PATCH] hw/i386/vmport: Drop superfluous parenthesis around function typedef

[Eduardo Habkost]

On Wed, Jun 03, 2020 at 07:04:26AM +0200, Philippe Mathieu-Daudé wrote:
> On 5/5/20 4:28 PM, Philippe Mathieu-Daudé wrote:
> > Drop superfluous parenthesis around VMPortReadFunc typedef
> > (added in d67f679d99, missed to remove when moved in e595112985).
> > 
> > Suggested-by: Richard Henderson <richard.henderson@linaro.org>
> > Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> > ---
> >  hw/i386/vmport.h | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/hw/i386/vmport.h b/hw/i386/vmport.h
> > index 47eda7a22b..a6624e3045 100644
> > --- a/hw/i386/vmport.h
> > +++ b/hw/i386/vmport.h
> > @@ -27,7 +27,7 @@
> >  
> >  #define TYPE_VMPORT "vmport"
> >  
> > -typedef uint32_t (VMPortReadFunc)(void *opaque, uint32_t address);
> > +typedef uint32_t VMPortReadFunc(void *opaque, uint32_t address);
> >  
> >  void vmport_register(unsigned char command, VMPortReadFunc *func, void *opaque);
> >  
> > 
> 
> ping?

Queued for 5.2.  Sorry for missing it.

-- 
Eduardo