diff options
| author | Sunny Kumar <sunkumar@redhat.com> | 2017-08-18 14:56:46 +0530 |
|---|---|---|
| committer | mohammed rafi kc <rkavunga@redhat.com> | 2017-09-04 16:48:36 +0000 |
| commit | d692f167662300ffad8d3ed44672182130025c75 (patch) | |
| tree | ea9dc5d98b39b9454a18267fee798e51226bc8d4 | |
| parent | 37e16afa4853dcb5df6d6d0c91a6afd960c3a901 (diff) | |
snapshot: clang compile warning
Warning type: NO_EFFECT
xlators/mgmt/glusterd/src/glusterd-snapshot.c:4004 : warning: comparison of
constant -1 with expression of type 'gf_boolean_t' (aka 'enum _gf_boolean')
is always false.
solution : "timestamp" was formerly declared as gf_boolean_t, now changed to
int.
Change-Id: Ie461537f06fe2971e2b09a428a22331808c41a13
BUG: 1335251
Signed-off-by: Sunny Kumar <sunkumar@redhat.com>
Reviewed-on: https://review.gluster.org/18062
Tested-by: Sunny Kumar
Smoke: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Amar Tumballi <amarts@redhat.com>
Reviewed-by: mohammed rafi kc <rkavunga@redhat.com>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-snapshot.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-snapshot.c b/xlators/mgmt/glusterd/src/glusterd-snapshot.c index 31f4d95f63d..d315b6582df 100644 --- a/xlators/mgmt/glusterd/src/glusterd-snapshot.c +++ b/xlators/mgmt/glusterd/src/glusterd-snapshot.c @@ -4001,7 +4001,7 @@ glusterd_handle_snapshot_create (rpcsvc_request_t *req, glusterd_op_t op, uuid_t *uuid_ptr = NULL; uuid_t tmp_uuid = {0}; int i = 0; - gf_boolean_t timestamp = _gf_false; + int timestamp = 0; char snap_volname[GD_VOLUME_NAME_MAX] = {0, }; char new_snapname[GLUSTERD_MAX_SNAP_NAME] = {0, }; char gmt_snaptime[GLUSTERD_MAX_SNAP_NAME] = {0, }; |