Details

    Description

      compile error using clang-3.8.0

      12:01:54 /source/storage/tokudb/PerconaFT/ft/txn/txn.cc:251:21: error: missing field '__count' initializer [-Werror,-Wmissing-field-initializers]
      12:01:54         .txn_lock = ZERO_MUTEX_INITIALIZER,
      12:01:54                     ^
      12:01:54 /source/storage/tokudb/PerconaFT/portability/toku_pthread.h:82:37: note: expanded from macro 'ZERO_MUTEX_INITIALIZER'
      12:01:54 # define ZERO_MUTEX_INITIALIZER {{{0}}}
      12:01:54                                     ^
      12:01:54 /source/storage/tokudb/PerconaFT/ft/txn/txn.cc:254:23: error: missing field '__count' initializer [-Werror,-Wmissing-field-initializers]
      12:01:54         .state_lock = ZERO_MUTEX_INITIALIZER,
      12:01:54                       ^
      12:01:54 /source/storage/tokudb/PerconaFT/portability/toku_pthread.h:82:37: note: expanded from macro 'ZERO_MUTEX_INITIALIZER'
      12:01:54 # define ZERO_MUTEX_INITIALIZER {{{0}}}
      12:01:54                                     ^
      12:01:54 2[ 30%] Built target sequence_embedded
      12:01:54  errors generated.
      

      Looks to me like ZERO_MUTEX_INITIALIZER should be defined to PTHREAD_MUTEX_INITIALIZER, at least on linux. The definition in pthread.h looks to be a struct of 0s anyway.

      Attachments

        Issue Links

          Activity

            cvicentiu recently fixed TokuDB+clang compilation failures in 5.5 (MDEV-10229), I don't know though if these ones have been covered. Assigning to check.

            cvicentiu, please modify Affects and Fix Version/s as needed.

            elenst Elena Stepanova added a comment - cvicentiu recently fixed TokuDB+clang compilation failures in 5.5 ( MDEV-10229 ), I don't know though if these ones have been covered. Assigning to check. cvicentiu , please modify Affects and Fix Version/s as needed.
            danblack Daniel Black added a comment - Dup of MDEV-10229 Thanks elenst and cvicentiu , https://github.com/MariaDB/server/commit/34a104ba0ccbe254100a235c04e4648136c9aa7e seems to cover the fix. Waiting for merge.

            Since MDEV-10229 fixes this issue, I've closed it as a duplicate.

            cvicentiu Vicențiu Ciorbaru added a comment - Since MDEV-10229 fixes this issue, I've closed it as a duplicate.

            People

              cvicentiu Vicențiu Ciorbaru
              danblack Daniel Black
              Votes:
              0 Vote for this issue
              Watchers:
              4 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.