Details
-
Task
-
Status: Closed (View Workflow)
-
Major
-
Resolution: Fixed
Description
InnoDB defines some functions that are not called at all. Other functions are called, but only from the same compilation unit.
Remove some function declarations and definitions, and add 'static' keywords. Some symbols must be kept for separately compiled tools, such as innochecksum.
Also, port the follow-up by Jon Olav Hauglid:
Bug#21153166: REMOVE UNUSED VARIABLES AND CONVERT GLOBAL SYMBOLS TO STATIC
Follow-up to
Bug#21141390: REMOVE UNUSED FUNCTIONS AND CONVERT GLOBAL SYMBOLS TO STATIC
but for variables instead of functions.
Was identified with the -Wmissing-variable-declarations
compiler warning option supported by Clang 3.6.
Attachments
Activity
Field | Original Value | New Value |
---|---|---|
Epic Link | MDEV-12265 [ 60576 ] |
Fix Version/s | 10.2.5 [ 22117 ] |
Component/s | Storage Engine - InnoDB [ 10129 ] |
Status | Open [ 1 ] | In Progress [ 3 ] |
Description |
InnoDB defines some functions that are not called at all. Other functions are called, but only from the same compilation unit.
Remove some function declarations and definitions, and add 'static' keywords. Some symbols must be kept for separately compiled tools, such as innochecksum. Also, port [the follow-up by Jon Olav Hauglid|https://github.com/mysql/mysql-server/commit/4718005900abdb28e054210be4cc77119e4539e9]: Bug#21153166: REMOVE UNUSED VARIABLES AND CONVERT GLOBAL SYMBOLS TO STATIC Follow-up to Bug#21141390: REMOVE UNUSED FUNCTIONS AND CONVERT GLOBAL SYMBOLS TO STATIC but for variables instead of functions. Was identified with the -Wmissing-variable-declarations compiler warning option supported by Clang 3.6. |
Assignee | Marko Mäkelä [ marko ] | Jan Lindström [ jplindst ] |
Status | In Progress [ 3 ] | In Review [ 10002 ] |
Assignee | Jan Lindström [ jplindst ] | Marko Mäkelä [ marko ] |
Status | In Review [ 10002 ] | Stalled [ 10000 ] |
Resolution | Fixed [ 1 ] | |
Status | Stalled [ 10000 ] | Closed [ 6 ] |
Workflow | MariaDB v3 [ 79987 ] | MariaDB v4 [ 133170 ] |
bb-10.2-marko~ and bb-10.2-marko