Skip to content

Commit 022e44c

Browse files
committed
Cleanup ReactChildren
1 parent 1aa9cc6 commit 022e44c

File tree

2 files changed

+12
-13
lines changed

2 files changed

+12
-13
lines changed

src/utils/ReactChildren.js

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -129,19 +129,19 @@ function mapChildren(children, func, context) {
129129
return mapResult;
130130
}
131131

132-
/**
133-
* Count the number of children that are typically specified as
134-
* `props.children`.
135-
*/
136-
137132
function forEachSingleChildDummy(traverseContext, child, name, i) {
138133
return null;
139134
}
140135

141-
function countChildren(children, context) {
142-
var numberOfChildren = traverseAllChildren(children, forEachSingleChildDummy,
143-
null);
144-
return numberOfChildren;
136+
/**
137+
* Count the number of children that are typically specified as
138+
* `props.children`.
139+
*
140+
* @param {?*} children Children tree container.
141+
* @return {number} The number of children.
142+
*/
143+
function countChildren(children, context) {
144+
return traverseAllChildren(children, forEachSingleChildDummy, null);
145145
}
146146

147147
var ReactChildren = {

src/utils/traverseAllChildren.js

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -183,12 +183,11 @@ var traverseAllChildrenImpl =
183183
* @return {!number} The number of children in this subtree.
184184
*/
185185
function traverseAllChildren(children, callback, traverseContext) {
186-
if (children !== null && children !== undefined) {
187-
return traverseAllChildrenImpl(children, '', 0, callback, traverseContext);
188-
}
189-
else {
186+
if (children == null) {
190187
return 0;
191188
}
189+
190+
return traverseAllChildrenImpl(children, '', 0, callback, traverseContext);
192191
}
193192

194193
module.exports = traverseAllChildren;

0 commit comments

Comments
 (0)