Skip to content

Commit

Permalink
use boolean to optimize streamUsage check
Browse files Browse the repository at this point in the history
  • Loading branch information
yaacovCR committed May 9, 2024
1 parent 92f9bb0 commit 96bf8b5
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions src/execution/execute.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1092,9 +1092,10 @@ async function completeAsyncIteratorValue(
];
let index = 0;
const streamUsage = getStreamUsage(exeContext, fieldGroup, path);
const hasStreamUsage = streamUsage !== undefined;
// eslint-disable-next-line no-constant-condition
while (true) {
if (streamUsage && index >= streamUsage.initialCount) {
if (hasStreamUsage && index >= streamUsage.initialCount) {
const returnFn = asyncIterator.return;
let streamRecord: SubsequentResultRecord | CancellableStreamRecord;
if (returnFn === undefined) {
Expand Down Expand Up @@ -1262,12 +1263,13 @@ function completeIterableValue(
];
let index = 0;
const streamUsage = getStreamUsage(exeContext, fieldGroup, path);
const hasStreamUsage = streamUsage !== undefined;
const iterator = items[Symbol.iterator]();
let iteration = iterator.next();
while (!iteration.done) {
const item = iteration.value;

if (streamUsage && index >= streamUsage.initialCount) {
if (hasStreamUsage && index >= streamUsage.initialCount) {
const streamRecord: SubsequentResultRecord = {
label: streamUsage.label,
path,
Expand Down

0 comments on commit 96bf8b5

Please sign in to comment.