From 0fa4177ed7e2e1ecacd729d92305c5406b767d4f Mon Sep 17 00:00:00 2001
From: Mikael Sand <msand@abo.fi>
Date: Sat, 18 Jan 2020 06:47:49 +0200
Subject: [PATCH] perf: optimize handling of font properties in G elements

---
 __tests__/__snapshots__/css.test.tsx.snap | 12 +++---------
 src/elements/G.tsx                        | 18 +++++++++++++-----
 2 files changed, 16 insertions(+), 14 deletions(-)

diff --git a/__tests__/__snapshots__/css.test.tsx.snap b/__tests__/__snapshots__/css.test.tsx.snap
index 79354d74..a486d502 100644
--- a/__tests__/__snapshots__/css.test.tsx.snap
+++ b/__tests__/__snapshots__/css.test.tsx.snap
@@ -155,13 +155,9 @@ exports[`supports CSS in style element 1`] = `
 </svg>"
   xmlns="http://www.w3.org/2000/svg"
 >
-  <RNSVGGroup
-    font={Object {}}
-  >
+  <RNSVGGroup>
     <RNSVGDefs />
-    <RNSVGGroup
-      font={Object {}}
-    >
+    <RNSVGGroup>
       <RNSVGRect
         fill={4294901760}
         height={200}
@@ -179,9 +175,7 @@ exports[`supports CSS in style element 1`] = `
         y={0}
       />
     </RNSVGGroup>
-    <RNSVGGroup
-      font={Object {}}
-    >
+    <RNSVGGroup>
       <RNSVGRect
         fill={4294901760}
         fillOpacity={0.3}
diff --git a/src/elements/G.tsx b/src/elements/G.tsx
index e7c7d318..65815df2 100644
--- a/src/elements/G.tsx
+++ b/src/elements/G.tsx
@@ -24,16 +24,24 @@ export default class G<P> extends Shape<P> {
   render() {
     const { props } = this;
     const prop = propsAndStyles(props);
+    const extractedProps = extractProps(prop, this);
+    const font = extractFont(prop);
+    if (hasProps(font)) {
+      extractedProps.font = font;
+    }
     return (
-      <RNSVGGroup
-        ref={this.refMethod}
-        {...extractProps(prop, this)}
-        font={extractFont(prop)}
-      >
+      <RNSVGGroup ref={this.refMethod} {...extractedProps}>
         {props.children}
       </RNSVGGroup>
     );
   }
 }
 
+const hasProps = (obj: {}) => {
+  for (let _ in obj) {
+    return true;
+  }
+  return false;
+};
+
 export const RNSVGGroup = requireNativeComponent('RNSVGGroup');