diff options
author | unknown <Kristofer.Pettersson@naruto.> | 2006-10-11 10:17:39 +0200 |
---|---|---|
committer | unknown <Kristofer.Pettersson@naruto.> | 2006-10-11 10:17:39 +0200 |
commit | c247bf9dc3c370c70dc763dcf361c85ee20035fd (patch) | |
tree | c3c6295c052cfb8644b61ca97122dcd1ffdd43d0 /include/my_time.h | |
parent | 910fe3921e8ba8a4562d5e9d318769e16babb110 (diff) | |
download | mariadb-git-c247bf9dc3c370c70dc763dcf361c85ee20035fd.tar.gz |
BUG#21811 Odd casting with date + INTERVAL arithmetic
- Type casting was not consequent, thus when adding a DATE type with
a WEEK interval the result tpe was DATETIME and not DATE as is the
norm.
- By changing the order of the date type enumerations the type casting
bug is resolved. To comply with the new order the array
interval_type_to_name needed to change accordingly.
include/my_time.h:
Changed enumeration order to make week interval appear in a more
logical order. This affects type casting when a date is added to an
interval.
sql/time.cc:
Moved string WEEK to match enumeration in interval_type
Diffstat (limited to 'include/my_time.h')
-rw-r--r-- | include/my_time.h | 20 |
1 files changed, 15 insertions, 5 deletions
diff --git a/include/my_time.h b/include/my_time.h index 3025b98a9c4..3e659630689 100644 --- a/include/my_time.h +++ b/include/my_time.h @@ -92,15 +92,25 @@ int my_date_to_str(const MYSQL_TIME *l_time, char *to); int my_datetime_to_str(const MYSQL_TIME *l_time, char *to); int my_TIME_to_str(const MYSQL_TIME *l_time, char *to); -/* - The following must be sorted so that simple intervals comes first. - (get_interval_value() depends on this) +/* + Available interval types used in any statement. + + 'interval_type' must be sorted so that simple intervals comes first, + ie year, quarter, month, week, day, hour, etc. The order based on + interval size is also important and the intervals should be kept in a + large to smaller order. (get_interval_value() depends on this) + + Note: If you change the order of elements in this enum you should fix + order of elements in 'interval_type_to_name' and 'interval_names' + arrays + + See also interval_type_to_name, get_interval_value, interval_names */ enum interval_type { - INTERVAL_YEAR, INTERVAL_QUARTER, INTERVAL_MONTH, INTERVAL_DAY, INTERVAL_HOUR, - INTERVAL_MINUTE, INTERVAL_WEEK, INTERVAL_SECOND, INTERVAL_MICROSECOND , + INTERVAL_YEAR, INTERVAL_QUARTER, INTERVAL_MONTH, INTERVAL_WEEK, INTERVAL_DAY, + INTERVAL_HOUR, INTERVAL_MINUTE, INTERVAL_SECOND, INTERVAL_MICROSECOND, INTERVAL_YEAR_MONTH, INTERVAL_DAY_HOUR, INTERVAL_DAY_MINUTE, INTERVAL_DAY_SECOND, INTERVAL_HOUR_MINUTE, INTERVAL_HOUR_SECOND, INTERVAL_MINUTE_SECOND, INTERVAL_DAY_MICROSECOND, INTERVAL_HOUR_MICROSECOND, |