From b69785b3b2d5839aa71c9dded7b037d7aab09007 Mon Sep 17 00:00:00 2001 From: demensky Date: Thu, 30 Jun 2022 15:51:01 +0300 Subject: [PATCH] refactor: removed useless thisArg from createFind --- src/internal/operators/find.ts | 10 +++------- src/internal/operators/findIndex.ts | 2 +- 2 files changed, 4 insertions(+), 8 deletions(-) diff --git a/src/internal/operators/find.ts b/src/internal/operators/find.ts index 9e3036779e..01b9e266e5 100644 --- a/src/internal/operators/find.ts +++ b/src/internal/operators/find.ts @@ -51,14 +51,10 @@ export function find(predicate: (value: T, index: number, source: Observable< * matches the condition. */ export function find(predicate: (value: T, index: number, source: Observable) => boolean): OperatorFunction { - return operate(createFind(predicate, undefined, 'value')); + return operate(createFind(predicate, 'value')); } -export function createFind( - predicate: (value: T, index: number, source: Observable) => boolean, - thisArg: any, - emit: 'value' | 'index' -) { +export function createFind(predicate: (value: T, index: number, source: Observable) => boolean, emit: 'value' | 'index') { const findIndex = emit === 'index'; return (source: Observable, subscriber: Subscriber) => { let index = 0; @@ -67,7 +63,7 @@ export function createFind( subscriber, (value) => { const i = index++; - if (predicate.call(thisArg, value, i, source)) { + if (predicate(value, i, source)) { subscriber.next(findIndex ? i : value); subscriber.complete(); } diff --git a/src/internal/operators/findIndex.ts b/src/internal/operators/findIndex.ts index debb72d1f9..3af7b606f4 100644 --- a/src/internal/operators/findIndex.ts +++ b/src/internal/operators/findIndex.ts @@ -48,5 +48,5 @@ export function findIndex(predicate: (value: T, index: number, source: Observ * first item that matches the condition. */ export function findIndex(predicate: (value: T, index: number, source: Observable) => boolean): OperatorFunction { - return operate(createFind(predicate, undefined, 'index')); + return operate(createFind(predicate, 'index')); }