Hoist makeStateUpdater into top of scope

Roblox · May 15, 2018 · d70b50b · d70b50b
1 parent 681cb25
commit d70b50b
Showing 1 changed file with 15 additions and 15 deletions.
diff --git a/lib/connect2.lua b/lib/connect2.lua
@@ -14,6 +14,21 @@ local function noop()
 	return nil
 end
 
+local function makeStateUpdater(store, mapStateToProps)
+	return function(nextProps, prevState, mappedStoreState)
+		if mappedStoreState == nil then
+			mappedStoreState = mapStateToProps(store:getState(), nextProps)
+		end
+
+		local propsForChild = join(nextProps, mappedStoreState, prevState.mappedStoreDispatch)
+
+		return {
+			mappedStoreState = mappedStoreState,
+			propsForChild = propsForChild,
+		}
+	end
+end
+
 --[[
 	mapStateToProps:
 		(storeState, props) -> partialProps
@@ -49,21 +64,6 @@ local function connect(mapStateToPropsOrThunk, mapDispatchToProps)
 			error(message, 2)
 		end
 
-		local function makeStateUpdater(store, mapStateToProps)
-			return function(nextProps, prevState, mappedStoreState)
-				if mappedStoreState == nil then
-					mappedStoreState = mapStateToProps(store:getState(), nextProps)
-				end
-
-				local propsForChild = join(nextProps, mappedStoreState, prevState.mappedStoreDispatch)
-
-				return {
-					mappedStoreState = mappedStoreState,
-					propsForChild = propsForChild,
-				}
-			end
-		end
-
 		local componentName = ("RoduxConnection(%s)"):format(tostring(innerComponent))
 
 		local outerComponent = Roact.Component:extend(componentName)