← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to drizzled/function/str/conv.cc

  • Committer: Padraig O'Sullivan
  • Date: 2009-06-29 17:24:02 UTC
  • mto: This revision was merged to the branch mainline in revision 1081.
  • Revision ID: osullivan.padraig@gmail.com-20090629172402-9c5n1kr7ry7xgau7
Removed the dependency on knowing the position of an I_S table in the
schema_tables array defined in show.cc. This issue crops up in
prepare_schema_table. An issue with my design is that it increases the time
complexity from O(1) to O(n) in numerous places to determine an I_S table to
work on since we don't have an explicit index into the array and instead
need to search by name. However, as n is the number of I_S tables and this
number is quite small (at the moment n is < 30), we don't see this causing
any issue. This design makes the code much more maintainable and easier to
understand. Previously, modifying anything to do with the I_S tables meant
having to tip-toe around the issue of hard-coded indexes into the
schema_tables array.

Show diffs side-by-side

added added

removed removed

Lines of Context:
drizzled/function/str/conv.cc
 
1
/* -*- mode: c++; c-basic-offset: 2; indent-tabs-mode: nil; -*-
 
2
 *  vim:expandtab:shiftwidth=2:tabstop=2:smarttab:
 
3
 *
 
4
 *  Copyright (C) 2008 Sun Microsystems
 
5
 *
 
6
 *  This program is free software; you can redistribute it and/or modify
 
7
 *  it under the terms of the GNU General Public License as published by
 
8
 *  the Free Software Foundation; version 2 of the License.
 
9
 *
 
10
 *  This program is distributed in the hope that it will be useful,
 
11
 *  but WITHOUT ANY WARRANTY; without even the implied warranty of
 
12
 *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 
13
 *  GNU General Public License for more details.
 
14
 *
 
15
 *  You should have received a copy of the GNU General Public License
 
16
 *  along with this program; if not, write to the Free Software
 
17
 *  Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
 
18
 */
 
19
 
 
20
#include <drizzled/server_includes.h>
 
21
#include CSTDINT_H
 
22
#include <drizzled/function/str/conv.h>
 
23
 
 
24
String *Item_func_conv::val_str(String *str)
 
25
{
 
26
  assert(fixed == 1);
 
27
  String *res= args[0]->val_str(str);
 
28
  char *endptr,ans[65],*ptr;
 
29
  int64_t dec;
 
30
  int from_base= (int) args[1]->val_int();
 
31
  int to_base= (int) args[2]->val_int();
 
32
  int err;
 
33
 
 
34
  if (args[0]->null_value || args[1]->null_value || args[2]->null_value ||
 
35
      abs(to_base) > 36 || abs(to_base) < 2 ||
 
36
      abs(from_base) > 36 || abs(from_base) < 2 || !(res->length()))
 
37
  {
 
38
    null_value= 1;
 
39
    return NULL;
 
40
  }
 
41
  null_value= 0;
 
42
  unsigned_flag= !(from_base < 0);
 
43
 
 
44
  if (from_base < 0)
 
45
    dec= my_strntoll(res->charset(), res->ptr(), res->length(),
 
46
                     -from_base, &endptr, &err);
 
47
  else
 
48
    dec= (int64_t) my_strntoull(res->charset(), res->ptr(), res->length(),
 
49
                                 from_base, &endptr, &err);
 
50
 
 
51
  ptr= int64_t2str(dec, ans, to_base);
 
52
  if (str->copy(ans, (uint32_t) (ptr-ans), default_charset()))
 
53
    return &my_empty_string;
 
54
  return str;
 
55
}
 
56
 
 
57