Drizzle

Merge lp:~stewart/drizzle/bug911638 into lp:~drizzle-trunk/drizzle/development

  1. bug911638
  2. Merge into development
Proposed by Stewart Smith
Status: Merged
Approved by: Brian Aker
Approved revision: 2479
Merged at revision: 2486
Proposed branch: lp:~stewart/drizzle/bug911638
Merge into: lp:~drizzle-trunk/drizzle/development
Diff against target: 12 lines (+1/-1)
1 file modified
drizzled/errmsg_print.cc (+1/-1)
To merge this branch: bzr merge lp:~stewart/drizzle/bug911638
Reviewer Review Type Date Requested Status
Drizzle Merge Team Pending
Review via email: mp+87435@code.launchpad.net

Description of the change

simple off-by-one that would lead to the (very) wrong error message in some situations.

To post a comment you must log in.
Revision history for this message
Stewart Smith (stewart) wrote :

http://jenkins.drizzle.org/view/Drizzle-param/job/drizzle-param/913/

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'drizzled/errmsg_print.cc'
2--- drizzled/errmsg_print.cc 2011-10-20 02:45:57 +0000
3+++ drizzled/errmsg_print.cc 2012-01-04 07:25:29 +0000
4@@ -74,7 +74,7 @@
5
6 if (not extra.empty())
7 {
8- if (message.at(message.size()) != ' ')
9+ if (message.at(message.size()-1) != ' ')
10 message+= " ";
11
12 message+= "'";