summaryrefslogtreecommitdiffstats
path: root/api/src/glfs.c
diff options
context:
space:
mode:
authorNiels de Vos <ndevos@redhat.com>2017-01-26 16:45:51 +0100
committerRaghavendra G <rgowdapp@redhat.com>2017-01-31 00:04:44 -0500
commitf2ca301bd741e3e3f076cd3f72fcd377bcef2a1a (patch)
treeb729d719b29a80faaee6de85ccfb90f235c9fbb3 /api/src/glfs.c
parent1a95fc3036db51b82b6a80952f0908bc2019d24a (diff)
refcount: typecast function for calling on free
All of the functions called to free the refcounted structure are doing a typecast from (void*) to their own type taht is being free'd. This really is not needed and the refcount interface is made a little simpler without the requirement of typecasting. With this small improvement in the API, all callers are updated too. Change-Id: I32473b6d1799f62861d4b2d78ea30c09e6c80ab1 BUG: 1416889 Signed-off-by: Niels de Vos <ndevos@redhat.com> Reviewed-on: https://review.gluster.org/16471 Smoke: Gluster Build System <jenkins@build.gluster.org> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> Reviewed-by: Xavier Hernandez <xhernandez@datalab.es> CentOS-regression: Gluster Build System <jenkins@build.gluster.org> Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com>
Diffstat (limited to 'api/src/glfs.c')
-rw-r--r--api/src/glfs.c10
1 files changed, 3 insertions, 7 deletions
diff --git a/api/src/glfs.c b/api/src/glfs.c
index 94a12c0be31..a1c8f6f54cb 100644
--- a/api/src/glfs.c
+++ b/api/src/glfs.c
@@ -583,16 +583,12 @@ pub_glfs_from_glfd (struct glfs_fd *glfd)
GFAPI_SYMVER_PUBLIC_DEFAULT(glfs_from_glfd, 3.4.0);
-void
-glfs_fd_destroy (void *data)
+static void
+glfs_fd_destroy (struct glfs_fd *glfd)
{
- struct glfs_fd *glfd = NULL;
-
- if (!data)
+ if (!glfd)
return;
- glfd = (struct glfs_fd *)data;
-
glfs_lock (glfd->fs, _gf_true);
{
list_del_init (&glfd->openfds);