svn commit: r187960 - in head/sys/fs: nwfs smbfs

Bjoern A. Zeeb bz at FreeBSD.org
Sat Jan 31 10:06:34 PST 2009


Author: bz
Date: Sat Jan 31 18:06:34 2009
New Revision: 187960
URL: http://svn.freebsd.org/changeset/base/187960

Log:
  After r186194 the *fs_strategy() functions always return 0.
  So we are no longer interested in the error returned from
  the *fs_doio() functions. With that we can remove the
  error variable as its value is unused now.
  
  Submitted by:	Christoph Mallon christoph.mallon at gmx.de

Modified:
  head/sys/fs/nwfs/nwfs_vnops.c
  head/sys/fs/smbfs/smbfs_vnops.c

Modified: head/sys/fs/nwfs/nwfs_vnops.c
==============================================================================
--- head/sys/fs/nwfs/nwfs_vnops.c	Sat Jan 31 17:36:22 2009	(r187959)
+++ head/sys/fs/nwfs/nwfs_vnops.c	Sat Jan 31 18:06:34 2009	(r187960)
@@ -784,7 +784,6 @@ static int nwfs_strategy (ap) 
 	struct buf *bp=ap->a_bp;
 	struct ucred *cr;
 	struct thread *td;
-	int error = 0;
 
 	NCPVNDEBUG("\n");
 	if (bp->b_flags & B_ASYNC)
@@ -801,7 +800,7 @@ static int nwfs_strategy (ap) 
 	 * otherwise just do it ourselves.
 	 */
 	if ((bp->b_flags & B_ASYNC) == 0 )
-		error = nwfs_doio(ap->a_vp, bp, cr, td);
+		(void)nwfs_doio(ap->a_vp, bp, cr, td);
 	return (0);
 }
 

Modified: head/sys/fs/smbfs/smbfs_vnops.c
==============================================================================
--- head/sys/fs/smbfs/smbfs_vnops.c	Sat Jan 31 17:36:22 2009	(r187959)
+++ head/sys/fs/smbfs/smbfs_vnops.c	Sat Jan 31 18:06:34 2009	(r187960)
@@ -850,7 +850,6 @@ smbfs_strategy (ap) 
 	struct buf *bp=ap->a_bp;
 	struct ucred *cr;
 	struct thread *td;
-	int error = 0;
 
 	SMBVDEBUG("\n");
 	if (bp->b_flags & B_ASYNC)
@@ -863,7 +862,7 @@ smbfs_strategy (ap) 
 		cr = bp->b_wcred;
 
 	if ((bp->b_flags & B_ASYNC) == 0 )
-		error = smbfs_doio(ap->a_vp, bp, cr, td);
+		(void)smbfs_doio(ap->a_vp, bp, cr, td);
 	return (0);
 }
 


More information about the svn-src-head mailing list