Merge lp://qastaging/~sinha/drizzle/trunk-bug-621856 into lp://qastaging/drizzle/7.0
Proposed by
Akash Sinha
Status: | Superseded |
---|---|
Proposed branch: | lp://qastaging/~sinha/drizzle/trunk-bug-621856 |
Merge into: | lp://qastaging/drizzle/7.0 |
Diff against target: |
213 lines (+49/-50) 2 files modified
drizzled/cursor.h (+46/-47) drizzled/sql_parse.cc (+3/-3) |
To merge this branch: | bzr merge lp://qastaging/~sinha/drizzle/trunk-bug-621856 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Stewart Smith (community) | Disapprove | ||
Vijay Samuel | Needs Fixing | ||
David Shrewsbury | Pending | ||
Review via email: mp+52931@code.qastaging.launchpad.net |
This proposal has been superseded by a proposal from 2011-03-15.
Description of the change
Using regex ,found bunch of functions with unused params in sql_parse.cc file and cursor.h file .
If the changes are fine,then i will start doing the same for rest of the files in drizzled directory.
To post a comment you must log in.
Unmerged revisions
- 2228. By Akash Sinha<email address hidden>
-
Removed functions with unused params
Hi Akash,
Good to see you submitting your first branch. Great job. There are coding standard violations in your branch. Please make it a point to adhere to all the coding guidelines inn the wiki page that I gave you.
I ll just point out some of the obvious one that I could see,
1) assignment operations should be as follows,
a= b; one space after the = and nonee before.
2) use two space indentations for new blocks of code.
3) braces must be written in new lines.
Please make it a point to fix all the above mentioned and re submit your merge proposal.
Cheers,
-Vijay