Skip to content

Fix an issue where defaultLinesDiffComputer does not pass in the timeout variable - #213035

Merged
Henning Dieterichs (hediet) merged 1 commit into
microsoft:mainfrom
CharlesHGong:CharlesHGong-patch-1
May 27, 2024
Merged

Henning Dieterichs (hediet) merged 1 commit into
microsoft:mainfrom
CharlesHGong:CharlesHGong-patch-1

Conversation

@CharlesHGong

@CharlesHGong Hanning Gong (Charles) (CharlesHGong) commented May 19, 2024

Copy link
Copy Markdown

Fixes #213034

This caller is missing to pass in the timeout variable.
Otherwise it won't respect the maxComputationTimeMs option that was passed in in computeDiff

The definition of the myersDiffingAlgorithm.compute looks like this

compute(seq1: ISequence, seq2: ISequence, timeout: ITimeout = InfiniteTimeout.instance)

@CharlesHGong

Copy link
Copy Markdown
Author

cc Henning Dieterichs (@hediet) the last editor of the line.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@hediet
Henning Dieterichs (hediet) enabled auto-merge (squash) May 27, 2024 16:37
@hediet Henning Dieterichs (hediet) added this to the May 2024 milestone May 27, 2024
@hediet
Henning Dieterichs (hediet) merged commit dccb86c into microsoft:main May 27, 2024
@microsoft Microsoft (microsoft) locked and limited conversation to collaborators Jul 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DefaultLinesDiffComputer did not pass timeout as an option

5 participants