diff options
author | Philip Withnall <philip.withnall@collabora.co.uk> | 2013-12-03 15:26:57 -0500 |
---|---|---|
committer | Matthew Barnes <mbarnes@redhat.com> | 2013-12-03 19:39:52 -0500 |
commit | f7b21d07f5bdc3a40979c89d75ffc40c13abeca2 (patch) | |
tree | b8f8a4d4a3b1d457f6f18efff7ace8ee44cc49ee | |
parent | 3df0d7608cb03375ae8bf365807bb64844e30c7c (diff) | |
download | evolution-data-server-f7b21d07f5bdc3a40979c89d75ffc40c13abeca2.tar.gz |
EBookBackendCache: Remove an unnecessary NULL check.
A newly-created EContact is never NULL, so the check is redundant.
It was confusing the static analyser.
https://bugzilla.gnome.org/719533
(cherry picked from commit 047c4470a554c6f4e9a5a83b414d2d65ef759436)
-rw-r--r-- | addressbook/libedata-book/e-book-backend-cache.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/addressbook/libedata-book/e-book-backend-cache.c b/addressbook/libedata-book/e-book-backend-cache.c index 105ef7736..00e02069b 100644 --- a/addressbook/libedata-book/e-book-backend-cache.c +++ b/addressbook/libedata-book/e-book-backend-cache.c @@ -215,7 +215,7 @@ e_book_backend_cache_get_contacts (EBookBackendCache *cache, if (vcard_str && !strncmp (vcard_str, "BEGIN:VCARD", 11)) { contact = e_contact_new_from_vcard (vcard_str); uid = e_contact_get_const (contact, E_CONTACT_UID); - if (contact && uid && *uid &&(query && e_book_backend_sexp_match_contact (sexp, contact))) + if (uid && *uid && (query && e_book_backend_sexp_match_contact (sexp, contact))) list = g_list_prepend (list, contact); else g_object_unref (contact); |