Skip to content

buffer: combine checking range of sourceStart in buf.copy - #47758

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
deokjinkim:230428_buffer_combine_check_sourcestart
May 7, 2023
Merged

buffer: combine checking range of sourceStart in buf.copy#47758
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
deokjinkim:230428_buffer_combine_check_sourcestart

Conversation

@deokjinkim

Copy link
Copy Markdown
Contributor

Merging 2 checking range of sourceStart into 1. Plus, add test case to increase coverage if sourceStart is greater than length of source.

Merging 2 checking range of sourceStart into 1. Plus,
add test case to increase coverage if sourceStart is greater
than length of source.
@nodejs-github-botnodejs-github-bot added buffer Issues and PRs related to the buffer subsystem. needs-ci PRs that need a full CI run. labels Apr 28, 2023
@deokjinkimdeokjinkim added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. request-ci Add this label to start a Jenkins CI on a PR. labels May 3, 2023
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label May 3, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@deokjinkimdeokjinkim added the commit-queue Add this label to land a pull request using GitHub Actions. label May 7, 2023
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label May 7, 2023
@nodejs-github-bot
nodejs-github-bot merged commit 2545019 into nodejs:mainMay 7, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 2545019

targos pushed a commit that referenced this pull request May 12, 2023
Merging 2 checking range of sourceStart into 1. Plus,
add test case to increase coverage if sourceStart is greater
than length of source.
PR-URL: #47758
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@targostargos mentioned this pull request May 15, 2023
danielleadams pushed a commit that referenced this pull request Jul 6, 2023
Merging 2 checking range of sourceStart into 1. Plus,
add test case to increase coverage if sourceStart is greater
than length of source.
PR-URL: #47758
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
MoLow pushed a commit to MoLow/node that referenced this pull request Jul 6, 2023
Merging 2 checking range of sourceStart into 1. Plus,
add test case to increase coverage if sourceStart is greater
than length of source.
PR-URL: nodejs#47758
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.bufferIssues and PRs related to the buffer subsystem.needs-ciPRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@deokjinkim@nodejs-github-bot@lpinca