Re: [PATCH] internal: Simplify STREQ_NULLABLE

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

 




On 12/09/2016 05:55 AM, Michal Privoznik wrote:
> Our STREQ_NULLABLE and STRNEQ_NULLABLE macros are too
> complicated. This was a result of some broken version of gcc.
> However, that is long gone and therefore we can simplify the
> macros.
> 
> Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
> ---
>  src/internal.h        |  4 ++--
>  tests/virstringtest.c | 61 +++++++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 63 insertions(+), 2 deletions(-)
> 

Going through older orphan patches...

Seems reasonable - the change being removing the '(a) ? (a) : ""' and
'(b) ? (b) : ""' from within the STR(N)EQ comparisons.

Any chance at knowing which compiler version "fixed" the issue with
strcmp and of course secondarily documenting or ensuring that version is
available in the "minimal requirements" for build?

ACK - for the code... Having that version is just extra just in case

John

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux