Opened 7 years ago

Closed 7 years ago

Last modified 5 years ago

#479 closed enhancement (fixed)

bind egg should support ___blob

Reported by: zbigniew 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 zbigniew)

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 zbigniew 7 years ago.
Add blob support

Download all attachments as: .zip

Change History (7)

comment:1 Changed 7 years ago by zbigniew

  • Description modified (diff)

Changed 7 years ago by zbigniew

Add blob support

comment:2 Changed 7 years ago by zbigniew

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 7 years ago by felix

  • Resolution set to fixed
  • Status changed from new to closed

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 7 years ago by zbigniew

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 6 years ago by felix

  • Milestone changed from 4.7.0 to 4.8.0

Milestone 4.7.0 deleted

comment:6 Changed 5 years ago by felix

  • Milestone changed from 4.8.0 to 4.9.0

Milestone 4.8.0 deleted

Note: See TracTickets for help on using tickets.