Skip to content

doc,sqlite: document entryPoint argument for loadExtension#63152

Merged
nodejs-github-bot merged 1 commit intonodejs:mainfrom
geeksilva97:doc/sqlite-load-extension-entry-point
May 8, 2026
Merged

doc,sqlite: document entryPoint argument for loadExtension#63152
nodejs-github-bot merged 1 commit intonodejs:mainfrom
geeksilva97:doc/sqlite-load-extension-entry-point

Conversation

@geeksilva97
Copy link
Copy Markdown
Contributor

@geeksilva97 geeksilva97 commented May 6, 2026

This PR adds the missing information about entryPoint argument in loadExtension.

@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/sqlite

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. sqlite Issues and PRs related to the SQLite subsystem. labels May 6, 2026
Signed-off-by: geeksilva97 <edigleyssonsilva@gmail.com>
@geeksilva97 geeksilva97 force-pushed the doc/sqlite-load-extension-entry-point branch from f7e273a to 4e10879 Compare May 6, 2026 14:00
@geeksilva97 geeksilva97 marked this pull request as ready for review May 6, 2026 14:23
@geeksilva97 geeksilva97 added the commit-queue Add this label to land a pull request using GitHub Actions. label May 7, 2026
Copy link
Copy Markdown
Member

@Renegade334 Renegade334 left a comment

Choose a reason for hiding this comment

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

LGTM, but I notice as a drive-by:

node/src/node_sqlite.cc

Lines 2435 to 2437 in a9b98b2

if (*entryPoint == nullptr) {
ToNamespacedPath(env, &entryPoint);
}

This isn't a path argument, so I don't think this should be here? (Actually, it's currently a no-op anyway, as it only runs when the argument doesn't exist!)

@geeksilva97
Copy link
Copy Markdown
Contributor Author

LGTM, but I notice as a drive-by:

node/src/node_sqlite.cc

Lines 2435 to 2437 in a9b98b2

if (*entryPoint == nullptr) {
ToNamespacedPath(env, &entryPoint);
}

This isn't a path argument, so I don't think this should be here? (Actually, it's currently a no-op anyway, as it only runs when the argument doesn't exist!)

Yeah, this should be removed.

@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label May 8, 2026
@nodejs-github-bot nodejs-github-bot merged commit 11740ef into nodejs:main May 8, 2026
33 of 34 checks passed
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Landed in 11740ef

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Issues and PRs related to the documentations. sqlite Issues and PRs related to the SQLite subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants