columns_with_metadata: access column metadata from prepared statement#1672
columns_with_metadata: access column metadata from prepared statement#1672gwenn merged 7 commits intorusqlite:masterfrom
Conversation
|
http://sqlite.org/c3ref/column_database_name.html
|
|
Yeah this was the reason I gated the impl behind |
http://sqlite.org/c3ref/column_decltype.html
=> |
|
I'm pretty sure most (all?) major distros build with this flag on even if it's technically an optional extra. |
https://github.com/gwenn/sqlite-jna/actions/runs/14461947474/job/40556051685
|
|
If you don't want to merge it that's fine :p |
if you don't want to make it mergeable that's funny :p |
|
Sorry I'm not trying to be glib, I just don't see what you want me to do here. Dynamically check whether the methods are present in the sqlite binary? |
|
You need to activate the Line 59 in 755bfa3 rusqlite/libsqlite3-sys/Cargo.toml Line 28 in 755bfa3 rusqlite/libsqlite3-sys/build.rs Lines 256 to 258 in 755bfa3 |
|
Ok I have given it a go, had a bit of trouble determining if it was working as expected though, perhaps you can review? |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1672 +/- ##
==========================================
+ Coverage 86.79% 86.86% +0.07%
==========================================
Files 58 58
Lines 11001 11088 +87
==========================================
+ Hits 9548 9632 +84
- Misses 1453 1456 +3 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
We might be able to have better coverage by using accessors instead of field. SELECT *, 1 FROM sqlite_schema;(any expression should work) |
|
And, if possible, some minimal doc. |
|
Thanks. |
|
Thank you for your help. |
|
Oops: |
|
#1675 1675 |
A simple binding for the column metadata functions:
sqlite3_column_database_name
sqlite3_column_table_name
sqlite3_column_origin_name
No docs or tests but I am happy to add them if you are interested in the PR.
Cheers,
Jack