Don't waste performance when there is no page break before function

Might improve #280.
bpampuch · May 5, 2015 · b31c381 · b31c381
1 parent dd7648f
commit b31c381
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/src/layoutBuilder.js b/src/layoutBuilder.js
@@ -54,6 +54,11 @@ LayoutBuilder.prototype.registerTableLayouts = function (tableLayouts) {
 LayoutBuilder.prototype.layoutDocument = function (docStructure, fontProvider, styleDictionary, defaultStyle, background, header, footer, images, watermark, pageBreakBeforeFct) {
 
   function addPageBreaksIfNecessary(linearNodeList, pages) {
+
+		if(!isFunction(pageBreakBeforeFct)){
+			return false;
+		}
+
     linearNodeList = _.reject(linearNodeList, function(node){
       return _.isEmpty(node.positions);
     });
@@ -100,12 +105,6 @@ LayoutBuilder.prototype.layoutDocument = function (docStructure, fontProvider, s
     });
   }
 
-  if(!isFunction(pageBreakBeforeFct)){
-    pageBreakBeforeFct = function(){
-      return false;
-    };
-  }
-
   this.docMeasure = new DocMeasure(fontProvider, styleDictionary, defaultStyle, this.imageMeasure, this.tableLayouts, images);