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:
|
|||
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.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
