Re: [PATCH 03/32] VFS/fsstack: cpp endif comments

2007-09-03 Thread Josef 'Jeff' Sipek 
On Mon, Sep 03, 2007 at 08:39:27AM +0200, Jan Engelhardt wrote:
> 
> On Sep 2 2007 22:20, Josef 'Jeff' Sipek wrote:
> >diff --git a/include/linux/fs_stack.h b/include/linux/fs_stack.h
> >index 6b52faf..28543ad 100644
> >--- a/include/linux/fs_stack.h
> >+++ b/include/linux/fs_stack.h
> >@@ -39,4 +39,4 @@ static inline void fsstack_copy_attr_times(struct inode 
> >*dest,
> > dest->i_ctime = src->i_ctime;
> > }
> > 
> >-#endif /* _LINUX_FS_STACK_H */
> >+#endif /* not _LINUX_FS_STACK_H */
> 
> I hardly think this changes a thing, and it even goes against
> the de-facto standard of a lot of other files.

Fair enough. I'm going to drop it.

Jeff.

-- 
My public GPG key can be found at
http://www.josefsipek.net/gpg/public-0xC7958FFE.txt
-
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 03/32] VFS/fsstack: cpp endif comments

2007-09-02 Thread Jan Engelhardt 

On Sep 2 2007 22:20, Josef 'Jeff' Sipek wrote:
>diff --git a/include/linux/fs_stack.h b/include/linux/fs_stack.h
>index 6b52faf..28543ad 100644
>--- a/include/linux/fs_stack.h
>+++ b/include/linux/fs_stack.h
>@@ -39,4 +39,4 @@ static inline void fsstack_copy_attr_times(struct inode 
>*dest,
>   dest->i_ctime = src->i_ctime;
> }
> 
>-#endif /* _LINUX_FS_STACK_H */
>+#endif /* not _LINUX_FS_STACK_H */

I hardly think this changes a thing, and it even goes against
the de-facto standard of a lot of other files.


Jan
-- 
-
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 03/32] VFS/fsstack: cpp endif comments

2007-09-02 Thread Josef 'Jeff' Sipek 
From: Erez Zadok <[EMAIL PROTECTED]>

Add comments to #endif's to help clarify code.

Signed-off-by: Erez Zadok <[EMAIL PROTECTED]>
Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]>
---
 include/linux/fs_stack.h |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/include/linux/fs_stack.h b/include/linux/fs_stack.h
index 6b52faf..28543ad 100644
--- a/include/linux/fs_stack.h
+++ b/include/linux/fs_stack.h
@@ -39,4 +39,4 @@ static inline void fsstack_copy_attr_times(struct inode *dest,
dest->i_ctime = src->i_ctime;
 }
 
-#endif /* _LINUX_FS_STACK_H */
+#endif /* not _LINUX_FS_STACK_H */
-- 
1.5.2.2.238.g7cbf2f2

-
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html