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.
This commit is contained in:
Patrick Walton 2019-03-07 11:23:44 -08:00
parent 640ac61b8d
commit 25558c2177
1 changed files with 3 additions and 1 deletions

View File

@ -156,7 +156,9 @@ impl BuiltSVG {
self.result_flags.insert(BuildResultFlags::UNSUPPORTED_CLIP_PATH_NODE); self.result_flags.insert(BuildResultFlags::UNSUPPORTED_CLIP_PATH_NODE);
} }
NodeKind::Defs { .. } => { NodeKind::Defs { .. } => {
self.result_flags.insert(BuildResultFlags::UNSUPPORTED_DEFS_NODE); if node.has_children() {
self.result_flags.insert(BuildResultFlags::UNSUPPORTED_DEFS_NODE);
}
} }
NodeKind::Filter(..) => { NodeKind::Filter(..) => {
self.result_flags.insert(BuildResultFlags::UNSUPPORTED_FILTER_NODE); self.result_flags.insert(BuildResultFlags::UNSUPPORTED_FILTER_NODE);