Message ID | 20200122072625.16487-1-luoshijie1@huawei.com |
---|---|
State | Superseded |
Headers | show |
Series | ext4,jbd2: fix comment and code style | expand |
On Wed 22-01-20 02:26:25, Shijie Luo wrote: > Fix comment and remove unnecessary blank. > > Signed-off-by: Shijie Luo <luoshijie1@huawei.com> Looks good. You can add: Reviewed-by: Jan Kara <jack@suse.cz> Honza > --- > fs/ext4/inline.c | 2 +- > fs/jbd2/transaction.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/fs/ext4/inline.c b/fs/ext4/inline.c > index 2fec62d764fa..a6695e1d246c 100644 > --- a/fs/ext4/inline.c > +++ b/fs/ext4/inline.c > @@ -849,7 +849,7 @@ static int ext4_da_convert_inline_data_to_extent(struct address_space *mapping, > > /* > * Prepare the write for the inline data. > - * If the the data can be written into the inode, we just read > + * If the data can be written into the inode, we just read > * the page and make it uptodate, and start the journal. > * Otherwise read the page, makes it dirty so that it can be > * handle in writepages(the i_disksize update is left to the > diff --git a/fs/jbd2/transaction.c b/fs/jbd2/transaction.c > index 27b9f9dee434..f7a9da75b160 100644 > --- a/fs/jbd2/transaction.c > +++ b/fs/jbd2/transaction.c > @@ -1595,7 +1595,7 @@ int jbd2_journal_dirty_metadata(handle_t *handle, struct buffer_head *bh) > * Allow this call even if the handle has aborted --- it may be part of > * the caller's cleanup after an abort. > */ > -int jbd2_journal_forget (handle_t *handle, struct buffer_head *bh) > +int jbd2_journal_forget(handle_t *handle, struct buffer_head *bh) > { > transaction_t *transaction = handle->h_transaction; > journal_t *journal; > -- > 2.19.1 >
On 2020/1/22 16:47, Jan Kara wrote: > On Wed 22-01-20 02:26:25, Shijie Luo wrote: >> Fix comment and remove unnecessary blank. >> >> Signed-off-by: Shijie Luo <luoshijie1@huawei.com> > Looks good. You can add: > > Reviewed-by: Jan Kara <jack@suse.cz> > > Honza Thanks for your review. >> --- >> fs/ext4/inline.c | 2 +- >> fs/jbd2/transaction.c | 2 +- >> 2 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/fs/ext4/inline.c b/fs/ext4/inline.c >> index 2fec62d764fa..a6695e1d246c 100644 >> --- a/fs/ext4/inline.c >> +++ b/fs/ext4/inline.c >> @@ -849,7 +849,7 @@ static int ext4_da_convert_inline_data_to_extent(struct address_space *mapping, >> >> /* >> * Prepare the write for the inline data. >> - * If the the data can be written into the inode, we just read >> + * If the data can be written into the inode, we just read >> * the page and make it uptodate, and start the journal. >> * Otherwise read the page, makes it dirty so that it can be >> * handle in writepages(the i_disksize update is left to the >> diff --git a/fs/jbd2/transaction.c b/fs/jbd2/transaction.c >> index 27b9f9dee434..f7a9da75b160 100644 >> --- a/fs/jbd2/transaction.c >> +++ b/fs/jbd2/transaction.c >> @@ -1595,7 +1595,7 @@ int jbd2_journal_dirty_metadata(handle_t *handle, struct buffer_head *bh) >> * Allow this call even if the handle has aborted --- it may be part of >> * the caller's cleanup after an abort. >> */ >> -int jbd2_journal_forget (handle_t *handle, struct buffer_head *bh) >> +int jbd2_journal_forget(handle_t *handle, struct buffer_head *bh) >> { >> transaction_t *transaction = handle->h_transaction; >> journal_t *journal; >> -- >> 2.19.1 >>
diff --git a/fs/ext4/inline.c b/fs/ext4/inline.c index 2fec62d764fa..a6695e1d246c 100644 --- a/fs/ext4/inline.c +++ b/fs/ext4/inline.c @@ -849,7 +849,7 @@ static int ext4_da_convert_inline_data_to_extent(struct address_space *mapping, /* * Prepare the write for the inline data. - * If the the data can be written into the inode, we just read + * If the data can be written into the inode, we just read * the page and make it uptodate, and start the journal. * Otherwise read the page, makes it dirty so that it can be * handle in writepages(the i_disksize update is left to the diff --git a/fs/jbd2/transaction.c b/fs/jbd2/transaction.c index 27b9f9dee434..f7a9da75b160 100644 --- a/fs/jbd2/transaction.c +++ b/fs/jbd2/transaction.c @@ -1595,7 +1595,7 @@ int jbd2_journal_dirty_metadata(handle_t *handle, struct buffer_head *bh) * Allow this call even if the handle has aborted --- it may be part of * the caller's cleanup after an abort. */ -int jbd2_journal_forget (handle_t *handle, struct buffer_head *bh) +int jbd2_journal_forget(handle_t *handle, struct buffer_head *bh) { transaction_t *transaction = handle->h_transaction; journal_t *journal;
Fix comment and remove unnecessary blank. Signed-off-by: Shijie Luo <luoshijie1@huawei.com> --- fs/ext4/inline.c | 2 +- fs/jbd2/transaction.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)