Uploaded image for project: 'MariaDB Server'
  1. MariaDB Server
  2. MDEV-21907

Enable -Wconversion for InnoDB and Mariabackup

Details

    Description

      To improve code quality and to avoid surprise build failures on Windows, we should fix any -Wconversion warnings that are reported by gcc or clang.

      gcc appears to be stricter than clang here; it will require explicit bitmask operations to suppress warnings about assignments to bit fields in structs.

      As a first step, we should enable -Wconversion in InnoDB and Mariabackup. This will require some headers in include/ and sql/ to be fixed as well. The Mariabackup build also includes some compilation units in libmysqld.

      Attachments

        Issue Links

          Activity

            marko Marko Mäkelä created issue -
            marko Marko Mäkelä made changes -
            Field Original Value New Value
            marko Marko Mäkelä made changes -
            Status Open [ 1 ] In Progress [ 3 ]
            marko Marko Mäkelä made changes -
            Description To improve code quality and to avoid surprise build failures on Windows, we should fix any {{-Wconversion}} warnings that are reported by {{gcc}} or {{clang}}.

            {{gcc}} appears to be stricter than {{clang}} here; it will require explicit bitmask operations to suppress warnings about assignments to bit fields in structs.

            As a first step, we should enable {{-Wconversion}} in InnoDB. This will require some headers in {{include/}} and {{sql/}} to be fixed as well.
            To improve code quality and to avoid surprise build failures on Windows, we should fix any {{-Wconversion}} warnings that are reported by {{gcc}} or {{clang}}.

            {{gcc}} appears to be stricter than {{clang}} here; it will require explicit bitmask operations to suppress warnings about assignments to bit fields in structs.

            As a first step, we should enable {{-Wconversion}} in InnoDB and Mariabackup. This will require some headers in {{include/}} and {{sql/}} to be fixed as well. The Mariabackup build also includes some compilation units in {{libmysqld}}.
            Summary Fix -Wconversion in InnoDB Enable -Wconversion for InnoDB and Mariabackup
            marko Marko Mäkelä made changes -
            marko Marko Mäkelä made changes -
            issue.field.resolutiondate 2020-03-12 18:10:32.0 2020-03-12 18:10:32.955
            marko Marko Mäkelä made changes -
            Fix Version/s 10.5.2 [ 24030 ]
            Fix Version/s 10.5 [ 23123 ]
            Resolution Fixed [ 1 ]
            Status In Progress [ 3 ] Closed [ 6 ]
            marko Marko Mäkelä made changes -
            wlad Vladislav Vaintroub made changes -
            Labels portability portability ulong
            marko Marko Mäkelä made changes -
            serg Sergei Golubchik made changes -
            Workflow MariaDB v3 [ 104738 ] MariaDB v4 [ 157411 ]

            People

              marko Marko Mäkelä
              marko Marko Mäkelä
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Git Integration

                  Error rendering 'com.xiplink.jira.git.jira_git_plugin:git-issue-webpanel'. Please contact your Jira administrators.