remove an unnecessary NULL guard on a call to free

Calling free on NULL is a well-defined no-op.

diff --git a/lib/common/htmlparse.y b/lib/common/htmlparse.y
index 4571c0c322d7d3880ca76d7a1f95eb896581aea3..1f1bf0d6e144747dca7e0d4311f7594b3cf084da 100644 (file)
--- a/lib/common/htmlparse.y
+++ b/lib/common/htmlparse.y
@@ -142,7 +142,7 @@ free_fspan(Dt_t* d, fspan* p, Dtdisc_t* ds)
        int i;
        ti = p->lp.items;
        for (i = 0; i < p->lp.nitems; i++) {
-           if (ti->str) free (ti->str);
+           free (ti->str);
            ti++;
        }
        free (p->lp.items);