diff options
| author | Marko Mäkelä <marko.makela@mariadb.com> | 2016-11-25 06:09:00 +0200 |
|---|---|---|
| committer | Marko Mäkelä <marko.makela@mariadb.com> | 2016-11-25 09:09:51 +0200 |
| commit | 8da33e3a868e7ab4472823f8f3bfab3d5cd1833e (patch) | |
| tree | b255d46a4c0afe7c2ad6396eda024989ebea65be /storage/xtradb/sync/sync0sync.cc | |
| parent | 1d8eafbeafbc4a77153d6ceb974dbbeff543e648 (diff) | |
| download | mariadb-git-8da33e3a868e7ab4472823f8f3bfab3d5cd1833e.tar.gz | |
MDEV-11349 (1/2) Fix some clang 4.0 warnings
In InnoDB and XtraDB functions that declare pointer parameters as nonnull,
remove nullness checks, because GCC would optimize them away anyway.
Use #ifdef instead of #if when checking for a configuration flag.
Clang says that left shifts of negative values are undefined.
So, use ~0U instead of ~0 in a number of macros.
Some functions that were defined as UNIV_INLINE were declared as
UNIV_INTERN. Consistently use the same type of linkage.
ibuf_merge_or_delete_for_page() could pass bitmap_page=NULL to
buf_page_print(), conflicting with the __attribute__((nonnull)).
Diffstat (limited to 'storage/xtradb/sync/sync0sync.cc')
| -rw-r--r-- | storage/xtradb/sync/sync0sync.cc | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/storage/xtradb/sync/sync0sync.cc b/storage/xtradb/sync/sync0sync.cc index 7458a052224..5c4b45eb3c0 100644 --- a/storage/xtradb/sync/sync0sync.cc +++ b/storage/xtradb/sync/sync0sync.cc @@ -223,9 +223,6 @@ UNIV_INTERN mysql_pfs_key_t mutex_list_mutex_key; /** Latching order checks start when this is set TRUE */ UNIV_INTERN ibool sync_order_checks_on = FALSE; -/** Number of slots reserved for each OS thread in the sync level array */ -static const ulint SYNC_THREAD_N_LEVELS = 10000; - /** Array for tracking sync levels per thread. */ typedef std::vector<sync_level_t> sync_arr_t; |