Markus Armbruster <arm...@redhat.com> writes: > Marc-André Lureau <marcandre.lur...@redhat.com> writes: > >> Use QLIST_FOREACH_ENTRY() to simplify the code and break earlier. >> >> Check that the QLit list has the same size as the qlist, this should >> ensure that we have an exact match when iterating over qlist for >> comparing the elements. >> >> Signed-off-by: Marc-André Lureau <marcandre.lur...@redhat.com> > > Reviewed-by: Markus Armbruster <arm...@redhat.com>
I'm updating the commit message so it continues to match the previous commit's message: qlit: Tighten QLit list vs QList comparison We check that all members of the QLit list are also in the QList. We neglect to check the other direction. Fix that. While there, use QLIST_FOREACH_ENTRY() to simplify the code and break the loop on the first mismatch. Hope that's okay with you.