* [PATCH] jbd2: clean up indentation issue, replace spaces with tab
@ 2018-11-23 16:40 Colin King
2018-11-26 13:56 ` Jan Kara
0 siblings, 1 reply; 3+ messages in thread
From: Colin King @ 2018-11-23 16:40 UTC (permalink / raw)
To: Theodore Ts'o, Jan Kara, linux-ext4; +Cc: kernel-janitors, linux-kernel
From: Colin Ian King <colin.king@canonical.com>
There is a statement that is indented with spaces, replace it with
a tab.
Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
fs/jbd2/transaction.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/jbd2/transaction.c b/fs/jbd2/transaction.c
index c0b66a7a795b..4af8ff17ad5c 100644
--- a/fs/jbd2/transaction.c
+++ b/fs/jbd2/transaction.c
@@ -910,7 +910,7 @@ do_get_write_access(handle_t *handle, struct journal_head *jh,
* this is the first time this transaction is touching this buffer,
* reset the modified flag
*/
- jh->b_modified = 0;
+ jh->b_modified = 0;
/*
* If the buffer is not journaled right now, we need to make sure it
--
2.19.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] jbd2: clean up indentation issue, replace spaces with tab
2018-11-23 16:40 [PATCH] jbd2: clean up indentation issue, replace spaces with tab Colin King
@ 2018-11-26 13:56 ` Jan Kara
2018-12-04 5:21 ` Theodore Y. Ts'o
0 siblings, 1 reply; 3+ messages in thread
From: Jan Kara @ 2018-11-26 13:56 UTC (permalink / raw)
To: Colin King
Cc: Theodore Ts'o, Jan Kara, linux-ext4, kernel-janitors, linux-kernel
On Fri 23-11-18 16:40:53, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> There is a statement that is indented with spaces, replace it with
> a tab.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
Looks good. You can add:
Reviewed-by: Jan Kara <jack@suse.cz>
Honza
> ---
> fs/jbd2/transaction.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/jbd2/transaction.c b/fs/jbd2/transaction.c
> index c0b66a7a795b..4af8ff17ad5c 100644
> --- a/fs/jbd2/transaction.c
> +++ b/fs/jbd2/transaction.c
> @@ -910,7 +910,7 @@ do_get_write_access(handle_t *handle, struct journal_head *jh,
> * this is the first time this transaction is touching this buffer,
> * reset the modified flag
> */
> - jh->b_modified = 0;
> + jh->b_modified = 0;
>
> /*
> * If the buffer is not journaled right now, we need to make sure it
> --
> 2.19.1
>
>
--
Jan Kara <jack@suse.com>
SUSE Labs, CR
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] jbd2: clean up indentation issue, replace spaces with tab
2018-11-26 13:56 ` Jan Kara
@ 2018-12-04 5:21 ` Theodore Y. Ts'o
0 siblings, 0 replies; 3+ messages in thread
From: Theodore Y. Ts'o @ 2018-12-04 5:21 UTC (permalink / raw)
To: Jan Kara; +Cc: Colin King, Jan Kara, linux-ext4, kernel-janitors, linux-kernel
On Mon, Nov 26, 2018 at 02:56:32PM +0100, Jan Kara wrote:
> On Fri 23-11-18 16:40:53, Colin King wrote:
> > From: Colin Ian King <colin.king@canonical.com>
> >
> > There is a statement that is indented with spaces, replace it with
> > a tab.
> >
> > Signed-off-by: Colin Ian King <colin.king@canonical.com>
>
> Looks good. You can add:
>
> Reviewed-by: Jan Kara <jack@suse.cz>
Thanks, applied.
- Ted
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-12-04 5:21 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-11-23 16:40 [PATCH] jbd2: clean up indentation issue, replace spaces with tab Colin King
2018-11-26 13:56 ` Jan Kara
2018-12-04 5:21 ` Theodore Y. Ts'o
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).