Skip to content

Commit 92d2dcc

Browse files
committed
Revert "Add helpful message about pooled classes"
This reverts commit e65f17b. This might have a perf impact so we're not going to go with it for now.
1 parent 4526f62 commit 92d2dcc

1 file changed

Lines changed: 12 additions & 29 deletions

File tree

src/utils/PooledClass.js

Lines changed: 12 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -20,14 +20,6 @@
2020

2121
var invariant = require('invariant');
2222

23-
var ABOUT_POOLING_MESSAGE = null;
24-
if (__DEV__) {
25-
ABOUT_POOLING_MESSAGE = (
26-
'This object is reused for performance reasons. If you\'re seeing this ' +
27-
'after logging an object, try logging individual properties.'
28-
);
29-
}
30-
3123
/**
3224
* Static poolers. Several custom versions for each potential number of
3325
* arguments. A completely generic pooler is easy to implement, but would
@@ -37,54 +29,46 @@ if (__DEV__) {
3729
*/
3830
var oneArgumentPooler = function(copyFieldsFrom) {
3931
var Klass = this;
40-
var instance;
4132
if (Klass.instancePool.length) {
42-
instance = Klass.instancePool.pop();
33+
var instance = Klass.instancePool.pop();
4334
Klass.call(instance, copyFieldsFrom);
35+
return instance;
4436
} else {
45-
instance = new Klass(copyFieldsFrom);
37+
return new Klass(copyFieldsFrom);
4638
}
47-
instance._ABOUT_POOLING = null;
48-
return instance;
4939
};
5040

5141
var twoArgumentPooler = function(a1, a2) {
5242
var Klass = this;
53-
var instance;
5443
if (Klass.instancePool.length) {
55-
instance = Klass.instancePool.pop();
44+
var instance = Klass.instancePool.pop();
5645
Klass.call(instance, a1, a2);
46+
return instance;
5747
} else {
58-
instance = new Klass(a1, a2);
48+
return new Klass(a1, a2);
5949
}
60-
instance._ABOUT_POOLING = null;
61-
return instance;
6250
};
6351

6452
var threeArgumentPooler = function(a1, a2, a3) {
6553
var Klass = this;
66-
var instance;
6754
if (Klass.instancePool.length) {
68-
instance = Klass.instancePool.pop();
55+
var instance = Klass.instancePool.pop();
6956
Klass.call(instance, a1, a2, a3);
57+
return instance;
7058
} else {
71-
instance = new Klass(a1, a2, a3);
59+
return new Klass(a1, a2, a3);
7260
}
73-
instance._ABOUT_POOLING = null;
74-
return instance;
7561
};
7662

7763
var fiveArgumentPooler = function(a1, a2, a3, a4, a5) {
7864
var Klass = this;
79-
var instance;
8065
if (Klass.instancePool.length) {
81-
instance = Klass.instancePool.pop();
66+
var instance = Klass.instancePool.pop();
8267
Klass.call(instance, a1, a2, a3, a4, a5);
68+
return instance;
8369
} else {
84-
instance = new Klass(a1, a2, a3, a4, a5);
70+
return new Klass(a1, a2, a3, a4, a5);
8571
}
86-
instance._ABOUT_POOLING = null;
87-
return instance;
8872
};
8973

9074
var standardReleaser = function(instance) {
@@ -96,7 +80,6 @@ var standardReleaser = function(instance) {
9680
if (instance.destructor) {
9781
instance.destructor();
9882
}
99-
instance._ABOUT_POOLING = ABOUT_POOLING_MESSAGE;
10083
if (Klass.instancePool.length < Klass.poolSize) {
10184
Klass.instancePool.push(instance);
10285
}

0 commit comments

Comments
 (0)