diff options
| author | Ravishankar N <ravishankar@redhat.com> | 2016-05-13 12:41:39 +0530 |
|---|---|---|
| committer | Niels de Vos <ndevos@redhat.com> | 2016-05-24 01:32:57 -0700 |
| commit | b128a8983376c70b6f9f35ed60d54404da314624 (patch) | |
| tree | 8d0dcaffa471cc8d52480c25b489cb8b3489ad58 /xlators/features/index/src/index.h | |
| parent | 2f278fa9b91b7e81e0d93ec44f486f12bdc3f50b (diff) | |
index: Fix compiler warning
Problem:
----------------------------
index.c:420:18: warning: comparison of unsigned enum expression < 0 is
always false [-Wtautological-compare]
if (type < 0 || type >= XATTROP_TYPE_END)
~~~~ ^ ~
1 warning generated.
----------------------------
Fix:
Add a -1 value in the enumeration and use that in if-statements.
Cherry picked from commit e221827c18ad4161a8e54ad67c9128ad234b8c03:
> BUG: 1335232
> Change-Id: I830a565b8d96d7b51ff775e94aa23dd567ffe0d9
> Signed-off-by: Niels de Vos <ndevos@redhat.com>
> Reviewed-on: http://review.gluster.org/14409
> Smoke: Gluster Build System <jenkins@build.gluster.com>
> Reviewed-by: Ravishankar N <ravishankar@redhat.com>
> CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
> Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com>
The backport prevens the need for commit 137bd83 to be backported.
Commit e221827 undoes the ugly type-casting.
Change-Id: I65911a05de5f8b5634f1e8cf359c38dad7d125aa
BUG: 1336268
Signed-off-by: Ravishankar N <ravishankar@redhat.com>
Signed-off-by: Niels de Vos <ndevow@redhat.com>
Reviewed-on: http://review.gluster.org/14346
Tested-by: Niels de Vos <ndevos@redhat.com>
Smoke: Gluster Build System <jenkins@build.gluster.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com>
Diffstat (limited to 'xlators/features/index/src/index.h')
| -rw-r--r-- | xlators/features/index/src/index.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/xlators/features/index/src/index.h b/xlators/features/index/src/index.h index 44e0039ff70..24fd293db70 100644 --- a/xlators/features/index/src/index.h +++ b/xlators/features/index/src/index.h @@ -27,6 +27,7 @@ typedef enum { } index_state_t; typedef enum { + XATTROP_TYPE_UNSET = -1, XATTROP, DIRTY, ENTRY_CHANGES, |