Drizzle

Merge lp://qastaging/~mordred/drizzle/some-set-var-cleanups into lp://qastaging/drizzle/7.0

  1. some-set-var-cleanups
  2. Merge into 7.0
Proposed by Monty Taylor
Status: Merged
Approved by: Monty Taylor
Approved revision: 2044
Merged at revision: 2048
Proposed branch: lp://qastaging/~mordred/drizzle/some-set-var-cleanups
Merge into: lp://qastaging/drizzle/7.0
Diff against target: 315 lines (+41/-55)
12 files modified
drizzled/function/get_system_var.cc (+1/-1)
drizzled/module/loader.cc (+0/-8)
drizzled/plugin.h (+0/-1)
drizzled/session.cc (+0/-1)
drizzled/set_var.cc (+13/-9)
drizzled/set_var.h (+7/-4)
drizzled/sql_lex.cc (+2/-0)
drizzled/sql_lex.h (+1/-1)
drizzled/sql_yacc.yy (+10/-7)
drizzled/statement/set_option.cc (+1/-2)
drizzled/sys_var.cc (+5/-19)
drizzled/sys_var.h (+1/-2)
To merge this branch: bzr merge lp://qastaging/~mordred/drizzle/some-set-var-cleanups
Reviewer Review Type Date Requested Status
Brian Aker Pending
Review via email: mp+45006@code.qastaging.launchpad.net

This proposal supersedes a proposal from 2010-12-30.

Description of the change

Some pre-refactor cleanup.

To post a comment you must log in.
Revision history for this message
Brian Aker (brianaker) wrote : Posted in a previous version of this proposal

Hades fail http://hudson.drizzle.org/view/Drizzle-build/job/drizzle-build-hades/992/console

review: Needs Fixing
Revision history for this message
Monty Taylor (mordred) wrote :

Fixed the osx failure. A vector wasn't being cleared when lex_start was re-called.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches