Inline options._skipEffects

Author: marvinhagemeister

src/index.js

@@ -14,7 +14,6 @@ const SHALLOW = { shallow: true };
 
 // components without names, kept as a hash for later comparison to return consistent UnnamedComponentXX names.
 const UNNAMED = [];
-const _skipEffects = '__s';
 
 const VOID_ELEMENTS = /^(area|base|br|col|embed|hr|img|input|link|meta|param|source|track|wbr)$/;
 
@@ -49,14 +48,20 @@ function renderToString(vnode, context, opts) {
 	context = context || {};
 	opts = opts || {};
 
-	const previousSkipEffects = options[_skipEffects];
-	options[_skipEffects] = true;
+	// Performance optimization: `renderToString` is synchronous and we
+	// therefore don't execute any effects. To do that we pass an empty
+	// array to `options._commit` (`__c`). But we can go one step further
+	// and avoid a lot of dirty checks and allocations by setting
+	// `options._skipEffects` (`__s`) too.
+	const previousSkipEffects = options.__s;
+	options.__s = true;
 
 	const res = _renderToString(vnode, context, opts);
+
 	// options._commit, we don't schedule any effects in this library right now,
 	// so we can pass an empty queue to this hook.
 	if (options.__c) options.__c(vnode, EMPTY_ARR);
-	options[_skipEffects] = previousSkipEffects;
+	options.__s = previousSkipEffects;
 	return res;
 }