Merge lp://qastaging/~linuxjedi/libdrizzle/5.1-fixes into lp://qastaging/libdrizzle
Proposed by
Andrew Hutchings
Status: | Merged | ||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Approved by: | Andrew Hutchings | ||||||||||||||||||||
Approved revision: | 116 | ||||||||||||||||||||
Merged at revision: | 115 | ||||||||||||||||||||
Proposed branch: | lp://qastaging/~linuxjedi/libdrizzle/5.1-fixes | ||||||||||||||||||||
Merge into: | lp://qastaging/libdrizzle | ||||||||||||||||||||
Diff against target: |
83 lines (+13/-5) 4 files modified
libdrizzle/field.cc (+0/-2) libdrizzle/result.cc (+1/-0) libdrizzle/row.cc (+11/-2) libdrizzle/statement_param.cc (+1/-1) |
||||||||||||||||||||
To merge this branch: | bzr merge lp://qastaging/~linuxjedi/libdrizzle/5.1-fixes | ||||||||||||||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Drizzle Trunk | Pending | ||
Review via email: mp+160823@code.qastaging.launchpad.net |
Description of the change
Fixes the following things:
1. Compile error in Ubuntu 13.04
2. Row allocation clearer and wastes less memory (at a later date we should probably make field_t a struct of data and length to improve this further)
3. Remove unnecessary push/pop in field code
To post a comment you must log in.