~drizzle-trunk/drizzle/development

  • Committer: Padraig
  • Date: 2009-02-13 08:26:21 UTC
  • mto: (873.2.28 devel)
  • Revision ID: posulliv@linux-lap-20090213082621-510acwp4j58z3qzc
Updated the fix for Bug#319796.

After chatting with Monty, we came to the conclusion that there is not much
of a need for enforcing a limit on the column alias length. Thus, we removed
the check that the length is less than MAX_ALIAS_LENGTH in Item::set_name.

We also removed all references to MAX_ALIAS_NAME in the code. It was only
used as fixed-length buffer in a few utility functions as a temporary place
for performing lower-case operations. We replaced these lower-case
  operations with the transform operations and used std::string's instead of
fixed-length buffers.

Thus, we were able to resolve this bug by actually removing some code :)
Filename Latest Rev Last Changed Committer Comment Size
..
dummy.c 815.1.1 16 years ago Monty Taylor Add timegm which is missing on Solaris. 1.6 KB Diff Download File
getopt.c 840.1.14 16 years ago Monty Taylor Merged from Mark. 31.3 KB Diff Download File
getopt.in.h 815.1.3 16 years ago Monty Taylor Added getopt workaround for broken solaris getopt. 7.8 KB Diff Download File
getopt1.c 815.1.3 16 years ago Monty Taylor Added getopt workaround for broken solaris getopt. 3.8 KB Diff Download File
getopt_int.h 815.1.3 16 years ago Monty Taylor Added getopt workaround for broken solaris getopt. 4.6 KB Diff Download File
Makefile.am 779.7.10 16 years ago Monty Taylor make distcheck cleanups. 5.4 KB Diff Download File
mktime.c 815.1.1 16 years ago Monty Taylor Add timegm which is missing on Solaris. 20.8 KB Diff Download File
time.in.h 815.1.1 16 years ago Monty Taylor Add timegm which is missing on Solaris. 3.5 KB Diff Download File
time_r.c 815.1.1 16 years ago Monty Taylor Add timegm which is missing on Solaris. 1.2 KB Diff Download File
timegm.c 815.1.6 16 years ago Monty Taylor Some tweaks to get timegm working on Solaris. 1.4 KB Diff Download File
unistd.in.h 815.1.3 16 years ago Monty Taylor Added getopt workaround for broken solaris getopt. 10.1 KB Diff Download File