Re: [PATCH 7/8] nfs: clean up function decls in nfs_page.h

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On May 19, 2014, at 12:36 PM, Jim Rees <rees@xxxxxxxxx> wrote:

> Consistency often trumps style guidelines, and I have no strong objection to
> this change. Just wanted to point it out.

Right and I appreciate you pointing it out. I mistakenly thought I was following the
guidelines!

So, I can see how it’d be useful to have names when using basic types or when
passing several arguments of the same type. Maybe we could just remove
super redundant names (i.e. “struct page *page”)?

On the other hand, maybe we don’t open this can of worms and I just drop this
patch from the patchset..

-dros

--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux