Skip to content

Commit 86db174

Browse files
committed
perf(line): prebind context of _changePolyState function to current instance rather than create an new function to reduce runtime memory cost (fix #20151)
1 parent 4d3cb2c commit 86db174

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/chart/line/LineView.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -624,6 +624,8 @@ class LineView extends ChartView {
624624

625625
this._symbolDraw = symbolDraw;
626626
this._lineGroup = lineGroup;
627+
628+
this._changePolyState = zrUtil.bind(this._changePolyState, this);
627629
}
628630

629631
render(seriesModel: LineSeriesModel, ecModel: GlobalModel, api: ExtensionAPI) {
@@ -885,9 +887,7 @@ class LineView extends ChartView {
885887
toggleHoverEmphasis(polygon, focus, blurScope, emphasisDisabled);
886888
}
887889

888-
const changePolyState = (toState: DisplayState) => {
889-
this._changePolyState(toState);
890-
};
890+
const changePolyState = this._changePolyState;
891891

892892
data.eachItemGraphicEl(function (el) {
893893
// Switch polyline / polygon state if element changed its state.

0 commit comments

Comments
 (0)