Skip to content

Commit 05a3534

Browse files
committed
linting
1 parent ad0ea84 commit 05a3534

File tree

4 files changed

+32
-29
lines changed

4 files changed

+32
-29
lines changed

src/Context.js

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,14 @@
11
import React from 'react';
2-
import MemoizeState from './Memoizer';
3-
import PropTypes from "prop-types";
2+
import PropTypes from 'prop-types';
3+
import MemoizedState from './Memoizer';
4+
45

56
export const MemoizeContext = ({ consumer: Consumer, selector, pure, children, ...rest }) => (
67
<Consumer {...rest}>
78
{values =>
8-
(<MemoizeState {...values} compute={selector} pure={pure}>
9+
(<MemoizedState {...values} compute={selector} pure={pure}>
910
{result => children(result)}
10-
</MemoizeState>)
11+
</MemoizedState>)
1112
}
1213
</Consumer>
1314
);
@@ -21,4 +22,4 @@ MemoizeContext.propTypes = {
2122

2223
MemoizeContext.defaultProps = {
2324
pure: true,
24-
};
25+
};

src/Memoizer.js

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import React, { Component } from 'react';
1+
import { Component } from 'react';
22
import PropTypes from 'prop-types';
33
import polyfill from 'react-lifecycles-compat';
44

@@ -17,23 +17,22 @@ const createMemoizer = (memoizationFunction) => {
1717
pure: false,
1818
};
1919

20-
state = {
21-
calculateResult: memoizationFunction(this.props.compute),
22-
};
23-
2420
static getDerivedStateFromProps(props, state) {
25-
const { calculateResult, result: oldResult } = state;
21+
const { calculateResult } = state;
2622
const { children, compute, pure, ...rest } = props;
2723
const result = state.calculateResult(rest);
28-
const changed = oldResult !== result;
2924
return {
3025
calculateResult,
3126
result,
3227
};
3328
}
3429

30+
state = {
31+
calculateResult: memoizationFunction(this.props.compute),
32+
};
33+
3534
shouldComponentUpdate(nextProps, nextState) {
36-
return !nextProps.pure || nextState.result!==this.state.result;
35+
return !nextProps.pure || nextState.result !== this.state.result;
3736
}
3837

3938
render() {
@@ -48,4 +47,5 @@ const createMemoizer = (memoizationFunction) => {
4847
export const MemoizeOne = createMemoizer(memoizeOne);
4948
export const MemoizeState = createMemoizer(memoizeState);
5049

51-
export default MemoizeState;
50+
/* eslint-disable */
51+
export default MemoizeState;

src/Waterflow.js

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,34 +1,36 @@
11
import React from 'react';
22
import PropTypes from 'prop-types';
33
import memoizeState from 'memoize-state';
4-
import polyfill from "react-lifecycles-compat";
4+
import polyfill from 'react-lifecycles-compat';
55

66
const genFlow = flow => flow.map(fn => memoizeState(fn));
77

88
export class MemoizedFlow extends React.Component {
99
static propTypes = {
10+
/* eslint-disable */
1011
input: PropTypes.object.isRequired,
12+
/* eslint-enable */
1113
flow: PropTypes.arrayOf(PropTypes.func).isRequired,
12-
children: PropTypes.func
13-
};
14-
15-
state = {
16-
flow: genFlow(this.props.flow)
14+
children: PropTypes.func.isRequired,
1715
};
1816

1917
static getDerivedStateFromProps(props, state) {
2018
return {
21-
value: state.flow.reduce((value, fn) => Object.assign({}, value, fn(value)), props.input)
22-
}
19+
value: state.flow.reduce((value, fn) => Object.assign({}, value, fn(value)), props.input),
20+
};
2321
}
2422

23+
state = {
24+
flow: genFlow(this.props.flow),
25+
};
26+
2527
shouldComponentUpdate(nextProps, nextState) {
2628
return nextState.value !== this.state.value;
2729
}
2830

2931
render() {
3032
return this.props.children(this.state.value);
3133
}
32-
};
34+
}
3335

34-
polyfill(MemoizedFlow);
36+
polyfill(MemoizedFlow);

src/index.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
1-
import MemoizeDefault, {MemoizeOne, MemoizeState} from './Memoizer'
2-
import {MemoizeContext} from "./Context";
3-
import {MemoizedFlow} from "./Waterflow";
1+
import MemoizeDefault, { MemoizeOne, MemoizeState } from './Memoizer';
2+
import { MemoizeContext } from './Context';
3+
import { MemoizedFlow } from './Waterflow';
44

55
export {
66
MemoizeOne,
77
MemoizeState,
88
MemoizeContext,
9-
MemoizedFlow
9+
MemoizedFlow,
1010
};
1111

12-
export default MemoizeDefault;
12+
export default MemoizeDefault;

0 commit comments

Comments
 (0)