Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Loadable extension #1362

Merged
merged 15 commits into from
Nov 12, 2023
Merged

Loadable extension #1362

merged 15 commits into from
Nov 12, 2023

Conversation

gwenn
Copy link
Collaborator

@gwenn gwenn commented Jul 9, 2023

otool -L target/debug/examples/libloadable_extension.dylib
...
	/usr/lib/libsqlite3.dylib (compatibility version 9.0.0, current version 321.3.0)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1292.100.5)

@codecov
Copy link

codecov bot commented Jul 9, 2023

Codecov Report

Attention: 27 lines in your changes are missing coverage. Please review.

Comparison is base (d7ac6a1) 81.33% compared to head (e8c35fc) 81.19%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1362      +/-   ##
==========================================
- Coverage   81.33%   81.19%   -0.14%     
==========================================
  Files          49       50       +1     
  Lines       10229    10276      +47     
==========================================
+ Hits         8320     8344      +24     
- Misses       1909     1932      +23     
Files Coverage Δ
libsqlite3-sys/src/error.rs 10.00% <ø> (ø)
src/error.rs 26.25% <ø> (ø)
src/lib.rs 84.19% <ø> (ø)
tests/deny_single_threaded_sqlite_config.rs 93.33% <100.00%> (+1.02%) ⬆️
src/inner_connection.rs 80.71% <0.00%> (-0.27%) ⬇️
src/trace.rs 99.06% <50.00%> (ø)
examples/load_extension.rs 0.00% <0.00%> (ø)
libsqlite3-sys/build.rs 64.74% <73.58%> (+0.14%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

gwenn added 7 commits July 9, 2023 15:53
`to_sqlite_error` needs `sqlite3_malloc`

```
sqlite> .log on
sqlite> .load target/debug/examples/libloadable_extension.so
Error: error during initialization: SQLite version mismatch: 3014000 < 3042000
```
@gwenn
Copy link
Collaborator Author

gwenn commented Aug 18, 2023

% cargo bloat --crates --example loadable_extension --features="loadable_extension modern_sqlite functions vtab trace"
...
24.3%  53.9% 378.5KiB libsqlite3_sys
16.2%  35.8% 251.6KiB std
 4.0%   8.8%  61.8KiB rusqlite
 0.3%   0.7%   5.0KiB bitflags
 0.2%   0.4%   3.1KiB atomic
 0.1%   0.1%     912B memchr
 0.0%   0.1%     496B [Unknown]
 0.0%   0.1%     368B loadable_extension
 0.0%   0.0%     320B hashbrown
 0.0%   0.0%     208B smallvec
 0.0%   0.0%     128B allocator_api2
45.2% 100.0% 702.3KiB .text section size, the file size is 1.5MiB

@gwenn gwenn marked this pull request as ready for review September 3, 2023 10:25
@gwenn gwenn requested a review from thomcc September 3, 2023 10:25
@gwenn
Copy link
Collaborator Author

gwenn commented Oct 7, 2023

@thomcc would you mind reviewing this PR ?

@gwenn
Copy link
Collaborator Author

gwenn commented Nov 10, 2023

@thomcc should I merge or wait ? (see #1412) Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant