Summary
bind egg should support ___blob
Metadata
- Id: 841dc75f5fcf911ed45fc8ded357b7cf5938793a
- Trac id: 479
- Type: enhancement
- Reporter: zbigniew
- Owner:
- Cc:
- Status: closed
- Component: extensions
- Estimated difficulty:
- Resolution: fixed
- Priority: minor
- Milestone: 4.9.0
- Version:
- Changetime: 2012-09-24 21:47:48 UTC
- Created: 2011-01-13 06:49:08 UTC
- Keywords: bind byte-vector wee green blobbie
Attachments
- 841dc75f5fcf911ed45fc8ded357b7cf5938793a/attachments/bind-blob.diff.txt
Description
bind supports ___byte_vector, which is deprecated. It should probably support ___blob in addition. I currently use ___byte_vector in sql-de-lite and would like to modernize that.
Changes and comments
[2011-01-13 06:50:38 UTC] zbigniew changed description
[2011-01-13 08:47:48 UTC] zbigniew attached bind-blob.diff.txt (description=Add __blob support)
[2011-01-13 08:50:22 UTC] zbigniew wrote:
Attached patch should do the trick. Though underlying byte-vector container is char*, I used void* to reflect the opacity of blobs.
[2011-01-19 11:41:55 UTC] felix changed status from new to closed
[2011-01-19 11:41:55 UTC] felix set resolution to fixed
[2011-01-19 11:41:55 UTC] felix wrote:
Thanks - patch is applied and 0.94 tagged.
Note that `___blob` is not compilable in C code yet - this needs a definition (just like `___byte_vector` in `chicken.h`). I have added such a definition in the "experimental" branch of the core repository.
[2011-01-19 19:05:46 UTC] zbigniew wrote:
Works fine without chicken.h patch when binding:
int sqlite3_bind_blob(sqlite3_stmt*, int, ___blob b, int n, void(*)(void*));
In what case is the chicken.h patch required?
[2011-06-01 09:00:41 UTC] felix changed milestone from 4.7.0 to 4.8.0
[2011-06-01 09:00:41 UTC] felix wrote:
Milestone 4.7.0 deleted
[2012-09-24 21:47:48 UTC] felix changed milestone from 4.8.0 to 4.9.0
[2012-09-24 21:47:48 UTC] felix wrote:
Milestone 4.8.0 deleted