[Qemu-devel] [PATCH V2 0/4] fix usage of bool

Jafar Abdi posted 4 patches 5 years, 1 month ago
Test docker-mingw@fedora passed
Test docker-clang@ubuntu passed
Test asan passed
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1553351197-14581-1-git-send-email-cafer.abdi@gmail.com
Maintainers: "Daniel P. Berrangé" <berrange@redhat.com>, Stefan Berger <stefanb@linux.ibm.com>, Thomas Huth <thuth@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, David Gibson <david@gibson.dropbear.id.au>, Laurent Vivier <lvivier@redhat.com>
authz/listfile.c         | 2 +-
hw/tpm/tpm_tis.c         | 4 ++--
tests/libqos/pci-pc.c    | 2 +-
tests/libqos/pci-spapr.c | 2 +-
4 files changed, 5 insertions(+), 5 deletions(-)
[Qemu-devel] [PATCH V2 0/4] fix usage of bool
Posted by Jafar Abdi 5 years, 1 month ago
Clean up wrong usage of FALSE and TRUE in places that use "bool" from stdbool.h.

FALSE and TRUE (with capital letters) are the constants defined by glib for
being used with the "gboolean" type of glib. But some parts of the code also use
TRUE and FALSE for variables that are declared as "bool" (the type from <stdbool.h>).

Jafar Abdi (4):
  authz: fix usage of bool in listfile.c
  tests/libqos: fix usage of bool in pci-pc.c
  tests/libqos: fix usage of bool in pci-spapr.c
  hw/tpm: fix usage of bool in tpm-tis.c

 authz/listfile.c         | 2 +-
 hw/tpm/tpm_tis.c         | 4 ++--
 tests/libqos/pci-pc.c    | 2 +-
 tests/libqos/pci-spapr.c | 2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

-- 
2.7.4


Re: [Qemu-devel] [PATCH V2 0/4] fix usage of bool
Posted by Stefan Hajnoczi 5 years, 1 month ago
On Sat, Mar 23, 2019 at 05:26:33PM +0300, Jafar Abdi wrote:
> Clean up wrong usage of FALSE and TRUE in places that use "bool" from stdbool.h.
> 
> FALSE and TRUE (with capital letters) are the constants defined by glib for
> being used with the "gboolean" type of glib. But some parts of the code also use
> TRUE and FALSE for variables that are declared as "bool" (the type from <stdbool.h>).
> 
> Jafar Abdi (4):
>   authz: fix usage of bool in listfile.c
>   tests/libqos: fix usage of bool in pci-pc.c
>   tests/libqos: fix usage of bool in pci-spapr.c
>   hw/tpm: fix usage of bool in tpm-tis.c
> 
>  authz/listfile.c         | 2 +-
>  hw/tpm/tpm_tis.c         | 4 ++--
>  tests/libqos/pci-pc.c    | 2 +-
>  tests/libqos/pci-spapr.c | 2 +-
>  4 files changed, 5 insertions(+), 5 deletions(-)
> 
> -- 
> 2.7.4
> 

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Re: [Qemu-devel] [PATCH V2 0/4] fix usage of bool
Posted by Stefan Hajnoczi 5 years, 1 month ago
On Sat, Mar 23, 2019 at 05:26:33PM +0300, Jafar Abdi wrote:
> Clean up wrong usage of FALSE and TRUE in places that use "bool" from stdbool.h.
> 
> FALSE and TRUE (with capital letters) are the constants defined by glib for
> being used with the "gboolean" type of glib. But some parts of the code also use
> TRUE and FALSE for variables that are declared as "bool" (the type from <stdbool.h>).

CCing qemu-trivial

> Jafar Abdi (4):
>   authz: fix usage of bool in listfile.c
>   tests/libqos: fix usage of bool in pci-pc.c
>   tests/libqos: fix usage of bool in pci-spapr.c
>   hw/tpm: fix usage of bool in tpm-tis.c
> 
>  authz/listfile.c         | 2 +-
>  hw/tpm/tpm_tis.c         | 4 ++--
>  tests/libqos/pci-pc.c    | 2 +-
>  tests/libqos/pci-spapr.c | 2 +-
>  4 files changed, 5 insertions(+), 5 deletions(-)
> 
> -- 
> 2.7.4
> 
Re: [Qemu-devel] [PATCH V2 0/4] fix usage of bool
Posted by Daniel P. Berrangé 4 years, 8 months ago
On Tue, Mar 26, 2019 at 08:18:14AM +0000, Stefan Hajnoczi wrote:
> On Sat, Mar 23, 2019 at 05:26:33PM +0300, Jafar Abdi wrote:
> > Clean up wrong usage of FALSE and TRUE in places that use "bool" from stdbool.h.
> > 
> > FALSE and TRUE (with capital letters) are the constants defined by glib for
> > being used with the "gboolean" type of glib. But some parts of the code also use
> > TRUE and FALSE for variables that are declared as "bool" (the type from <stdbool.h>).
> 
> CCing qemu-trivial

Ping qemu-trivial - this was seems to have been missed, though
patches 2 & 3 got merged via ppc tree meanwhile. The other two
are still valid.


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|