From 25558c217755b2ca41ca6b84fbf277441baac24c Mon Sep 17 00:00:00 2001
From: Patrick Walton <pcwalton@mimiga.net>
Date: Thu, 7 Mar 2019 11:23:44 -0800
Subject: [PATCH] Only emit warnings for non-empty `<defs>` nodes.

Inkscape likes to put empty `<defs>` nodes in the file, which don't do
anything, so we technically support them.
---
 svg/src/lib.rs | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/svg/src/lib.rs b/svg/src/lib.rs
index 31c7da94..ba96f02b 100644
--- a/svg/src/lib.rs
+++ b/svg/src/lib.rs
@@ -156,7 +156,9 @@ impl BuiltSVG {
                 self.result_flags.insert(BuildResultFlags::UNSUPPORTED_CLIP_PATH_NODE);
             }
             NodeKind::Defs { .. } => {
-                self.result_flags.insert(BuildResultFlags::UNSUPPORTED_DEFS_NODE);
+                if node.has_children() {
+                    self.result_flags.insert(BuildResultFlags::UNSUPPORTED_DEFS_NODE);
+                }
             }
             NodeKind::Filter(..) => {
                 self.result_flags.insert(BuildResultFlags::UNSUPPORTED_FILTER_NODE);