Hi On Fri, Sep 7, 2018 at 12:26 PM Thomas Huth <th...@redhat.com> wrote: > > On 2018-09-07 09:59, Marc-André Lureau wrote: > > A char** variant of strcmp(). > > > > Signed-off-by: Marc-André Lureau <marcandre.lur...@redhat.com> > > --- > > include/qemu/cutils.h | 12 ++++++++++++ > > util/cutils.c | 5 +++++ > > 2 files changed, 17 insertions(+) > > > > diff --git a/include/qemu/cutils.h b/include/qemu/cutils.h > > index 47aaa3b0b9..0941639f36 100644 > > --- a/include/qemu/cutils.h > > +++ b/include/qemu/cutils.h > > @@ -169,4 +169,16 @@ bool test_buffer_is_zero_next_accel(void); > > int uleb128_encode_small(uint8_t *out, uint32_t n); > > int uleb128_decode_small(const uint8_t *in, uint32_t *n); > > > > +/** > > + * qemu_pstrcmp: > > + * @str1: a pointer to a C string > > + * @str2: a pointer to a C string > > + * > > + * Compares *str1 and *str2 with g_strcmp0(). > > + * > > + * Returns: an integer less than, equal to, or greater than zero, if > > + * *str1 is <, == or > than *str2 . > > + */ > > Maybe add a comment that str1 or str2 can also be NULL?
*str1 or *str2 (not str1 or str2). ok > > Reviewed-by: Thomas Huth <th...@redhat.com> > -- Marc-André Lureau