Date: Sun, 23 Apr 2006 20:45:24 GMT From: John Birrell <jb@FreeBSD.org> To: Perforce Change Reviews <perforce@freebsd.org> Subject: PERFORCE change 95949 for review Message-ID: <200604232045.k3NKjOhE060534@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=95949 Change 95949 by jb@jb_freebsd2 on 2006/04/23 20:44:35 Remove the debugging printfs. This code all seems to be working correctly. Affected files ... .. //depot/projects/dtrace/src/contrib/opensolaris/lib/libdtrace/common/dt_provider.c#4 edit Differences ... ==== //depot/projects/dtrace/src/contrib/opensolaris/lib/libdtrace/common/dt_provider.c#4 (text) ==== @@ -678,7 +678,6 @@ * If none is found and an explicit probe ID was specified, discover * that specific probe and cache its description and arguments. */ -printf("%s:%s(%d) call dt_provider_lookup in existing cache\n",__FUNCTION__,__FILE__,__LINE__); if ((pvp = dt_provider_lookup(dtp, pdp->dtpd_provider)) != NULL) { size_t keylen = dt_probe_keylen(pdp); char *key = dt_probe_key(pdp, alloca(keylen)); @@ -713,10 +712,8 @@ if ((m = dtrace_probe_iter(dtp, pdp, dt_probe_desc, &pd)) < 0) return (NULL); /* dt_errno is set for us */ -printf("%s:%s(%d) call dt_provider_lookup\n",__FUNCTION__,__FILE__,__LINE__); if ((pvp = dt_provider_lookup(dtp, pd.dtpd_provider)) == NULL) return (NULL); /* dt_errno is set for us */ -printf("%s:%s(%d) got a provider\n",__FUNCTION__,__FILE__,__LINE__); /* * If more than one probe was matched, then do not report probe @@ -765,15 +762,9 @@ * the real attributes. Otherwise grab the static declaration. */ if (pd.dtpd_id != DTRACE_IDNONE) -{ -printf("%s:%s(%d) call dt_probe_discover\n",__FUNCTION__,__FILE__,__LINE__); prp = dt_probe_discover(pvp, &pd); -} else -{ -printf("%s:%s(%d) call dt_probe_lookup\n",__FUNCTION__,__FILE__,__LINE__); prp = dt_probe_lookup(pvp, pd.dtpd_name); -} if (prp == NULL) return (NULL); /* dt_errno is set for us */ @@ -803,7 +794,6 @@ pip->dtp_arga = pap->dtpa_args; pip->dtp_argv = prp->pr_argv; pip->dtp_argc = prp->pr_argc; -printf("%s:%s(%d) return the probe pointer\n",__FUNCTION__,__FILE__,__LINE__); return (prp); }
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200604232045.k3NKjOhE060534>