Thread (7 messages) 7 messages, 4 authors, 2012-04-24

Re: [PATCH] xfs_fsr: Get the last mount on a specific mount point

From: Christoph Hellwig <hch@infradead.org>
Date: 2012-02-17 17:26:41

On Wed, Feb 15, 2012 at 10:31:43AM -0200, Carlos Maiolino wrote:
I would change a little this sentence, looks a little confusing at least for me.
How about:

		/* If we found an entry based on the device name, make sure we
		 * stat the mount point given by mtab and ensure it's accessible
		 * before using it.
		 */
Agreed, I'll redo it.
quoted
+			if (S_ISLNK(sb.st_mode)) {
+				struct stat64 sb2;
Particularly, I don't like variable declarations between statements, but I'm not
too experienced to say what's good or not :)
The idea is to keep variable declarations as local as possible.

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help