Merge lp://staging/~linuxjedi/libdrizzle/5.1-prep-stmt-cleanup into lp://staging/libdrizzle

Proposed by Andrew Hutchings
Status: Merged
Approved by: Andrew Hutchings
Approved revision: 77
Merged at revision: 75
Proposed branch: lp://staging/~linuxjedi/libdrizzle/5.1-prep-stmt-cleanup
Merge into: lp://staging/libdrizzle
Diff against target: 1193 lines (+885/-100)
13 files modified
libdrizzle-5.1/constants.h (+0/-9)
libdrizzle-5.1/return.h (+2/-0)
libdrizzle-5.1/statement.h (+47/-16)
libdrizzle/common.h (+1/-0)
libdrizzle/error.cc (+2/-0)
libdrizzle/include.am (+2/-0)
libdrizzle/statement.cc (+19/-68)
libdrizzle/statement_local.h (+65/-0)
libdrizzle/statement_param.cc (+559/-0)
libdrizzle/structs.h (+8/-1)
tests/unit/include.am (+5/-0)
tests/unit/statement.c (+18/-6)
tests/unit/statement_char.c (+157/-0)
To merge this branch: bzr merge lp://staging/~linuxjedi/libdrizzle/5.1-prep-stmt-cleanup
Reviewer Review Type Date Requested Status
Andrew Hutchings Needs Fixing
Review via email: mp+141510@code.staging.launchpad.net

Description of the change

Adds a JDBC like interface to libdrizzle. Now to set query params there are functions like:

drizzle_return_t drizzle_stmt_set_long(drizzle_stmt_st *stmt, uint16_t param_num, uint32_t value, bool is_unsigned);

and

drizzle_return_t drizzle_stmt_set_string(drizzle_stmt_st *stmt, uint16_t param_num, char *value, size_t length);

And to get result params:

uint32_t drizzle_stmt_get_long(drizzle_stmt_st *stmt, uint16_t column_number, drizzle_return_t *ret_ptr);

and

const char *drizzle_stmt_get_string(drizzle_stmt_st *stmt, uint16_t column_number, size_t *len, drizzle_return_t *ret_ptr);

To post a comment you must log in.
Revision history for this message
Andrew Hutchings (linuxjedi) wrote :

Need to change the names from 'long' to 'int'

review: Needs Fixing
77. By Andrew Hutchings

Change long to int in function names

78. By Andrew Hutchings

Fix statement_char test case

79. By Andrew Hutchings

Using wrong source for column_count

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes:
to status/vote changes: