1 files changed, 25 insertions, 17 deletions

diff --git a/cddl/contrib/opensolaris/tools/ctf/cvt/tdata.c b/cddl/contrib/opensolaris/tools/ctf/cvt/tdata.c
index 32d8482..4a0cc79 100644
--- a/cddl/contrib/opensolaris/tools/ctf/cvt/tdata.c
+++ b/cddl/contrib/opensolaris/tools/ctf/cvt/tdata.c
@@ -174,9 +174,10 @@ tdesc_namecmp(void *arg1, void *arg2)
 	return (!streq(tdp1->t_name, tdp2->t_name));
 }
 
+#if defined(sun)
 /*ARGSUSED1*/
-int
-tdesc_print(void *data, void *private)
+static int
+tdesc_print(void *data, void *private __unused)
 {
 	tdesc_t *tdp = data;
 
@@ -184,6 +185,7 @@ tdesc_print(void *data, void *private)
 
 	return (1);
 }
+#endif
 
 static void
 free_intr(tdesc_t *tdp)
@@ -247,39 +249,42 @@ static void (*free_cbs[])(tdesc_t *) = {
 };
 
 /*ARGSUSED1*/
-static int
-tdesc_free_cb(tdesc_t *tdp, void *private)
+static void
+tdesc_free_cb(void *arg, void *private __unused)
 {
+	tdesc_t *tdp = arg;
 	if (tdp->t_name)
 		free(tdp->t_name);
 	if (free_cbs[tdp->t_type])
 		free_cbs[tdp->t_type](tdp);
 	free(tdp);
 
-	return (1);
+	return;
 }
 
 void
 tdesc_free(tdesc_t *tdp)
 {
-	(void) tdesc_free_cb(tdp, NULL);
+	tdesc_free_cb(tdp, NULL);
 }
 
 static int
-tdata_label_cmp(labelent_t *le1, labelent_t *le2)
+tdata_label_cmp(void *arg1, void *arg2)
 {
+	labelent_t *le1 = arg1;
+	labelent_t *le2 = arg2;
 	return (le1->le_idx - le2->le_idx);
 }
 
 void
-tdata_label_add(tdata_t *td, char *label, int idx)
+tdata_label_add(tdata_t *td, const char *label, int idx)
 {
 	labelent_t *le = xmalloc(sizeof (*le));
 
 	le->le_name = xstrdup(label);
 	le->le_idx = (idx == -1 ? td->td_nextid - 1 : idx);
 
-	slist_add(&td->td_labels, le, (int (*)())tdata_label_cmp);
+	slist_add(&td->td_labels, le, tdata_label_cmp);
 }
 
 static int
@@ -304,8 +309,10 @@ tdata_label_top(tdata_t *td)
 }
 
 static int
-tdata_label_find_cb(labelent_t *le, labelent_t *tmpl)
+tdata_label_find_cb(void *arg1, void *arg2)
 {
+	labelent_t *le = arg1;
+	labelent_t *tmpl = arg2;
 	return (streq(le->le_name, tmpl->le_name));
 }
 
@@ -323,7 +330,7 @@ tdata_label_find(tdata_t *td, char *label)
 	let.le_name = label;
 
 	if (!(ret = (labelent_t *)list_find(td->td_labels, &let,
-	    (int (*)())tdata_label_find_cb)))
+	    tdata_label_find_cb)))
 		return (-1);
 
 	return (ret->le_idx);
@@ -351,8 +358,9 @@ tdata_label_newmax(tdata_t *td, int newmax)
 
 /*ARGSUSED1*/
 static void
-tdata_label_free_cb(labelent_t *le, void *private)
+tdata_label_free_cb(void *arg, void *private __unused)
 {
+	labelent_t *le = arg;
 	if (le->le_name)
 		free(le->le_name);
 	free(le);
@@ -361,7 +369,7 @@ tdata_label_free_cb(labelent_t *le, void *private)
 void
 tdata_label_free(tdata_t *td)
 {
-	list_free(td->td_labels, (void (*)())tdata_label_free_cb, NULL);
+	list_free(td->td_labels, tdata_label_free_cb, NULL);
 	td->td_labels = NULL;
 }
 
@@ -391,8 +399,8 @@ tdata_new(void)
 void
 tdata_free(tdata_t *td)
 {
-	hash_free(td->td_iihash, (void (*)())iidesc_free, NULL);
-	hash_free(td->td_layouthash, (void (*)())tdesc_free_cb, NULL);
+	hash_free(td->td_iihash, iidesc_free, NULL);
+	hash_free(td->td_layouthash, tdesc_free_cb, NULL);
 	hash_free(td->td_idhash, NULL, NULL);
 	list_free(td->td_fwdlist, NULL, NULL);
 
@@ -408,7 +416,7 @@ tdata_free(tdata_t *td)
 
 /*ARGSUSED1*/
 static int
-build_hashes(tdesc_t *ctdp, tdesc_t **ctdpp, void *private)
+build_hashes(tdesc_t *ctdp, tdesc_t **ctdpp __unused, void *private)
 {
 	tdata_t *td = private;
 
@@ -465,7 +473,7 @@ tdata_merge(tdata_t *td1, tdata_t *td2)
 	td2->td_fwdlist = NULL;
 
 	slist_merge(&td1->td_labels, td2->td_labels,
-	    (int (*)())tdata_label_cmp);
+	    tdata_label_cmp);
 	td2->td_labels = NULL;
 
 	/* free the td2 hashes (data is now part of td1) */