From 1a62681e58eb99764da3619a987724840129e05c Mon Sep 17 00:00:00 2001
From: VladimirTechMan <VladimirTechMan@gmail.com>
Date: Tue, 15 Jan 2019 21:39:40 +0300
Subject: [PATCH] When rendering multi-line texts, check text decoration flags
 once

A small improvement to avoid re-testing the presence of underline
and line-through attributes in the textDecoration property on each
line of the text being rendered.
---
 src/shapes/Text.ts | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/shapes/Text.ts b/src/shapes/Text.ts
index abc2ba47..7707128f 100644
--- a/src/shapes/Text.ts
+++ b/src/shapes/Text.ts
@@ -137,9 +137,11 @@ export class Text extends Shape {
       align = this.align(),
       totalWidth = this.getWidth(),
       letterSpacing = this.letterSpacing(),
-      textDecoration = this.textDecoration(),
       fill = this.fill(),
       fontSize = this.fontSize(),
+      textDecoration = this.textDecoration(),
+      shouldUnderline = textDecoration.indexOf('underline') !== -1,
+      shouldLineThrough = textDecoration.indexOf('line-through') !== -1,
       n;
 
     context.setAttr('font', this._getContextFont());
@@ -181,7 +183,7 @@ export class Text extends Shape {
         context.translate((totalWidth - width - padding * 2) / 2, 0);
       }
 
-      if (textDecoration.indexOf('underline') !== -1) {
+      if (shouldUnderline) {
         context.save();
         context.beginPath();
 
@@ -200,7 +202,7 @@ export class Text extends Shape {
         context.stroke();
         context.restore();
       }
-      if (textDecoration.indexOf('line-through') !== -1) {
+      if (shouldLineThrough) {
         context.save();
         context.beginPath();
         context.moveTo(0, 0);