On Wed 17-01-18 04:58:32, Sidong Yang wrote: > Fix error reported by checkpatch.pl about space prohibited before > that close parenthesis ')'. > > Signed-off-by: Sidong Yang <realwakka@xxxxxxxxx> Sorry, but this is just making checkpatch happy without looking too much at the result. The indentation of FANOTIFY_INIT_ALL_EVENT_F_BITS definition is such that braces are aligned with | operators. And, although it is somewhat non-standard, it looks nice and better to me than the alternative suggested by checkpatch. So won't apply this patch. Honza > --- > fs/notify/fanotify/fanotify_user.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fs/notify/fanotify/fanotify_user.c b/fs/notify/fanotify/fanotify_user.c > index d0d4bc4c4b70..291fe6800bce 100644 > --- a/fs/notify/fanotify/fanotify_user.c > +++ b/fs/notify/fanotify/fanotify_user.c > @@ -38,7 +38,7 @@ > #define FANOTIFY_INIT_ALL_EVENT_F_BITS ( \ > O_ACCMODE | O_APPEND | O_NONBLOCK | \ > __O_SYNC | O_DSYNC | O_CLOEXEC | \ > - O_LARGEFILE | O_NOATIME ) > + O_LARGEFILE | O_NOATIME) > > extern const struct fsnotify_ops fanotify_fsnotify_ops; > > -- > 2.11.0 > -- Jan Kara <jack@xxxxxxxx> SUSE Labs, CR
