diff options
author | Sergei Petrunia <sergey@mariadb.com> | 2023-02-15 12:03:12 +0300 |
---|---|---|
committer | Sergei Petrunia <sergey@mariadb.com> | 2023-02-15 12:03:12 +0300 |
commit | 10a974adc9b250431cac44555c7aa9354e5840df (patch) | |
tree | ceb40f857edb82caf29a4f41f919fc9709f199f2 /mysys | |
parent | f74bb51b30df03cf21aca040901089ed27821762 (diff) | |
parent | e8c7222ba3add52665e425740860325866b2aef5 (diff) | |
download | mariadb-git-10a974adc9b250431cac44555c7aa9354e5840df.tar.gz |
Merge 11.0-selectivity into 11.0
Diffstat (limited to 'mysys')
-rw-r--r-- | mysys/array.c | 43 | ||||
-rw-r--r-- | mysys/mf_radix.c | 2 | ||||
-rw-r--r-- | mysys/mf_sort.c | 2 | ||||
-rw-r--r-- | mysys/my_getopt.c | 2 |
4 files changed, 43 insertions, 6 deletions
diff --git a/mysys/array.c b/mysys/array.c index 6e871ee6343..02a54d44656 100644 --- a/mysys/array.c +++ b/mysys/array.c @@ -92,7 +92,7 @@ my_bool init_dynamic_array2(PSI_memory_key psi_key, DYNAMIC_ARRAY *array, my_bool insert_dynamic(DYNAMIC_ARRAY *array, const void * element) { void *buffer; - if (array->elements == array->max_element) + if (unlikely(array->elements == array->max_element)) { /* Call only when necessary */ if (!(buffer=alloc_dynamic(array))) return TRUE; @@ -102,7 +102,42 @@ my_bool insert_dynamic(DYNAMIC_ARRAY *array, const void * element) buffer=array->buffer+(array->elements * array->size_of_element); array->elements++; } - memcpy(buffer,element,(size_t) array->size_of_element); + memcpy(buffer, element, array->size_of_element); + return FALSE; +} + + +/* Fast version of appending to dynamic array */ + +void init_append_dynamic(DYNAMIC_ARRAY_APPEND *append, + DYNAMIC_ARRAY *array) +{ + append->array= array; + append->pos= array->buffer + array->elements * array->size_of_element; + append->end= array->buffer + array->max_element * array->size_of_element; +} + + +my_bool append_dynamic(DYNAMIC_ARRAY_APPEND *append, + const void *element) +{ + DYNAMIC_ARRAY *array= append->array; + size_t size_of_element= array->size_of_element; + if (unlikely(append->pos == append->end)) + { + void *buffer; + if (!(buffer=alloc_dynamic(array))) + return TRUE; + append->pos= (uchar*)buffer + size_of_element; + append->end= array->buffer + array->max_element * size_of_element; + memcpy(buffer, element, size_of_element); + } + else + { + array->elements++; + memcpy(append->pos, element, size_of_element); + append->pos+= size_of_element; + } return FALSE; } @@ -281,7 +316,7 @@ my_bool allocate_dynamic(DYNAMIC_ARRAY *array, size_t max_elements) void get_dynamic(DYNAMIC_ARRAY *array, void *element, size_t idx) { - if (idx >= array->elements) + if (unlikely(idx >= array->elements)) { DBUG_PRINT("warning",("To big array idx: %d, array size is %d", idx,array->elements)); @@ -306,7 +341,7 @@ void delete_dynamic(DYNAMIC_ARRAY *array) /* Just mark as empty if we are using a static buffer */ - if (!(array->malloc_flags & MY_INIT_BUFFER_USED) && array->buffer) + if (array->buffer && !(array->malloc_flags & MY_INIT_BUFFER_USED)) my_free(array->buffer); array->buffer= 0; diff --git a/mysys/mf_radix.c b/mysys/mf_radix.c index 8f044cf9b29..89a3c4ac6ce 100644 --- a/mysys/mf_radix.c +++ b/mysys/mf_radix.c @@ -26,7 +26,7 @@ /* Radixsort */ -my_bool radixsort_is_appliccable(uint n_items, size_t size_of_element) +my_bool radixsort_is_applicable(uint n_items, size_t size_of_element) { return size_of_element <= 20 && n_items >= 1000 && n_items < 100000; } diff --git a/mysys/mf_sort.c b/mysys/mf_sort.c index 24e875b813e..79bdfdd23aa 100644 --- a/mysys/mf_sort.c +++ b/mysys/mf_sort.c @@ -23,7 +23,7 @@ void my_string_ptr_sort(uchar *base, uint items, size_t size) #if INT_MAX > 65536L uchar **ptr=0; - if (radixsort_is_appliccable(items, size) && + if (radixsort_is_applicable(items, size) && (ptr= (uchar**) my_malloc(PSI_NOT_INSTRUMENTED, items * sizeof(char*),MYF(0)))) { diff --git a/mysys/my_getopt.c b/mysys/my_getopt.c index 9f13ca7a424..d1b2c599372 100644 --- a/mysys/my_getopt.c +++ b/mysys/my_getopt.c @@ -897,7 +897,9 @@ static int setval(const struct my_option *opts, void *value, char *argument, goto ret; }; } + validate_value(opts->name, argument, option_file); + DBUG_RETURN(0); ret: |