Opened 14 years ago

Closed 14 years ago

Last modified 12 years ago

#479 closed enhancement (fixed)

bind egg should support ___blob

Reported by: Jim Ursetto Owned by:
Priority: minor Milestone: 4.9.0
Component: extensions Version:
Keywords: bind byte-vector wee green blobbie Cc:
Estimated difficulty:

Description (last modified by Jim Ursetto)

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.

Attachments (1)

bind-blob.diff.txt (2.5 KB) - added by Jim Ursetto 14 years ago.
Add blob support

Download all attachments as: .zip

Change History (7)

comment:1 Changed 14 years ago by Jim Ursetto

Description: modified (diff)

Changed 14 years ago by Jim Ursetto

Attachment: bind-blob.diff.txt added

Add blob support

comment:2 Changed 14 years ago by Jim Ursetto

Attached patch should do the trick. Though underlying byte-vector container is char*, I used void* to reflect the opacity of blobs.

comment:3 Changed 14 years ago by felix winkelmann

Resolution: fixed
Status: newclosed

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.

comment:4 Changed 14 years ago by Jim Ursetto

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?

comment:5 Changed 14 years ago by felix winkelmann

Milestone: 4.7.04.8.0

Milestone 4.7.0 deleted

comment:6 Changed 12 years ago by felix winkelmann

Milestone: 4.8.04.9.0

Milestone 4.8.0 deleted

Note: See TracTickets for help on using tickets.