Details
-
Task
-
Status: Closed (View Workflow)
-
Major
-
Resolution: Fixed
-
None
Description
During a review for MDEV-13995 Sanja suggested to have a separate class as an output parameter for val_raw_xxx(), e.g.:
bool val_raw_native(THD *thd, Raw *to); |
instead of:
bool val_raw_native(THD *thd, String *to); |
Passing a String looks redundant, as val_raw_xxx() does not need a character set.
Under terms of this task we'll:
- Extract the character-set-unaware part of String into a new class Binary_string
- Derive String from Binary_string and Charset
Then, while cleaning up the patch for MDEV-13995, we'll introduce a new class Raw and also derive it from Binary_string.
Attachments
Issue Links
- blocks
-
MDEV-13995 MAX(timestamp) returns a wrong result near DST change
- Closed
- is blocked by
-
MDEV-17907 Class Static_binary_string
- Closed
- relates to
-
MDEV-7063 Split String into logical components
- Stalled