add prep stmt column name lookup system
Bug #1094843 reported by
Andrew Hutchings
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Drizzle Client & Protocol Library |
Fix Released
|
Medium
|
Andrew Hutchings | ||
Bug Description
at the moment all _set functions require a column number. We actually have the column names as part of the prepare function. We should use this as a lookup for name -> number. Maybe using function overloading?
Related branches
lp:~linuxjedi/libdrizzle/5.1-api-fixes
- Drizzle Trunk: Pending requested
-
Diff: 1469 lines (+359/-161)40 files modifiedcli/drizzle_binlogs.c (+3/-3)
docs/api/binlog.rst (+2/-2)
docs/api/connection.rst (+2/-3)
docs/api/constants.rst (+47/-2)
docs/api/query.rst (+2/-2)
docs/api/statement.rst (+56/-1)
docs/api/structs.rst (+1/-35)
docs/examples/binlog.rst (+1/-1)
libdrizzle-5.1/binlog.h (+2/-2)
libdrizzle-5.1/column.h (+1/-1)
libdrizzle-5.1/conn.h (+1/-1)
libdrizzle-5.1/conn_client.h (+2/-3)
libdrizzle-5.1/query.h (+1/-1)
libdrizzle-5.1/return.h (+1/-0)
libdrizzle-5.1/statement.h (+19/-1)
libdrizzle-5.1/verbose.h (+1/-2)
libdrizzle/binlog.cc (+5/-5)
libdrizzle/column.cc (+2/-2)
libdrizzle/column.h (+1/-1)
libdrizzle/command.cc (+3/-3)
libdrizzle/conn.cc (+11/-8)
libdrizzle/drizzle.cc (+1/-2)
libdrizzle/drizzle_local.h (+4/-4)
libdrizzle/error.cc (+1/-0)
libdrizzle/handshake.cc (+7/-7)
libdrizzle/pack.cc (+13/-13)
libdrizzle/pack.h (+13/-13)
libdrizzle/query.cc (+7/-7)
libdrizzle/statement.cc (+12/-12)
libdrizzle/statement_local.h (+3/-0)
libdrizzle/statement_param.cc (+105/-0)
libdrizzle/structs.h (+9/-9)
tests/unit/column.c (+2/-2)
tests/unit/hex.c (+1/-1)
tests/unit/insert_id.c (+2/-2)
tests/unit/query.c (+2/-2)
tests/unit/row.c (+2/-2)
tests/unit/statement.c (+8/-2)
tests/unit/statement_char.c (+1/-2)
tests/unit/unbuffered_query.c (+2/-2)
| Changed in libdrizzle: | |
| status: | New → In Progress |
| importance: | Undecided → Medium |
| assignee: | nobody → Andrew Hutchings (linuxjedi) |
| milestone: | none → 5.1.1 |
| Changed in libdrizzle: | |
| status: | In Progress → Fix Released |
To post a comment you must log in.
