Merge lp://qastaging/~linuxjedi/libdrizzle/5.1-malloc-replace into lp://qastaging/libdrizzle
Proposed by
Andrew Hutchings
| Status: | Merged | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Approved by: | Andrew Hutchings | ||||||||
| Approved revision: | 94 | ||||||||
| Merged at revision: | 91 | ||||||||
| Proposed branch: | lp://qastaging/~linuxjedi/libdrizzle/5.1-malloc-replace | ||||||||
| Merge into: | lp://qastaging/libdrizzle | ||||||||
| Diff against target: |
735 lines (+180/-159) 12 files modified
libdrizzle-5.1/constants.h (+4/-3) libdrizzle/binlog.cc (+1/-1) libdrizzle/column.cc (+3/-22) libdrizzle/command.cc (+1/-1) libdrizzle/common.h (+1/-0) libdrizzle/conn.cc (+24/-4) libdrizzle/drizzle.cc (+3/-51) libdrizzle/field.cc (+1/-1) libdrizzle/handshake.cc (+2/-2) libdrizzle/result.cc (+5/-37) libdrizzle/statement.cc (+21/-29) libdrizzle/structs.h (+114/-8) |
||||||||
| To merge this branch: | bzr merge lp://qastaging/~linuxjedi/libdrizzle/5.1-malloc-replace | ||||||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Drizzle Trunk | Pending | ||
|
Review via email:
|
|||
Description of the change
* Switches malloc for new/delete
* Make structs initialize (for new)
* Change fixed 32K net buffer to an auto-growing variable net buffer 1MB -> 1GB
o Starts at 1MB
o If packet > buffer size (buffer is full) double buffer size
o If buffer size >= 1GB bail with error (instead of trying to recv 0 bytes as before)
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
