[PATCH] ceph: convert comma to semicolon

Chen Ni posted 1 patch 1 year, 5 months ago
fs/ceph/dir.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] ceph: convert comma to semicolon
Posted by Chen Ni 1 year, 5 months ago
Replace a comma between expression statements by a semicolon.

Signed-off-by: Chen Ni <nichen@iscas.ac.cn>
---
 fs/ceph/dir.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/ceph/dir.c b/fs/ceph/dir.c
index 5aadc56e0cc0..18c72b305858 100644
--- a/fs/ceph/dir.c
+++ b/fs/ceph/dir.c
@@ -1589,7 +1589,7 @@ void __ceph_dentry_dir_lease_touch(struct ceph_dentry_info *di)
 	}
 
 	spin_lock(&mdsc->dentry_list_lock);
-	__dentry_dir_lease_touch(mdsc, di),
+	__dentry_dir_lease_touch(mdsc, di);
 	spin_unlock(&mdsc->dentry_list_lock);
 }
 
-- 
2.25.1
Re: [PATCH] ceph: convert comma to semicolon
Posted by Xiubo Li 1 year, 5 months ago
On 7/9/24 14:44, Chen Ni wrote:
> Replace a comma between expression statements by a semicolon.
>
> Signed-off-by: Chen Ni <nichen@iscas.ac.cn>
> ---
>   fs/ceph/dir.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/ceph/dir.c b/fs/ceph/dir.c
> index 5aadc56e0cc0..18c72b305858 100644
> --- a/fs/ceph/dir.c
> +++ b/fs/ceph/dir.c
> @@ -1589,7 +1589,7 @@ void __ceph_dentry_dir_lease_touch(struct ceph_dentry_info *di)
>   	}
>   
>   	spin_lock(&mdsc->dentry_list_lock);
> -	__dentry_dir_lease_touch(mdsc, di),
> +	__dentry_dir_lease_touch(mdsc, di);
>   	spin_unlock(&mdsc->dentry_list_lock);
>   }
>   
Reviewed-by: Xiubo Li <xiubli@redhat.com>
Re: [PATCH] ceph: convert comma to semicolon
Posted by Ilya Dryomov 1 year, 5 months ago
On Wed, Jul 10, 2024 at 9:18 AM Xiubo Li <xiubli@redhat.com> wrote:
>
>
> On 7/9/24 14:44, Chen Ni wrote:
> > Replace a comma between expression statements by a semicolon.
> >
> > Signed-off-by: Chen Ni <nichen@iscas.ac.cn>
> > ---
> >   fs/ceph/dir.c | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/fs/ceph/dir.c b/fs/ceph/dir.c
> > index 5aadc56e0cc0..18c72b305858 100644
> > --- a/fs/ceph/dir.c
> > +++ b/fs/ceph/dir.c
> > @@ -1589,7 +1589,7 @@ void __ceph_dentry_dir_lease_touch(struct ceph_dentry_info *di)
> >       }
> >
> >       spin_lock(&mdsc->dentry_list_lock);
> > -     __dentry_dir_lease_touch(mdsc, di),
> > +     __dentry_dir_lease_touch(mdsc, di);
> >       spin_unlock(&mdsc->dentry_list_lock);
> >   }
> >
> Reviewed-by: Xiubo Li <xiubli@redhat.com>

Applied.

Thanks,

                Ilya