Hi Miklos,
Thank you for your mail. Please see my message below.
bingjingc <bingjingc@xxxxxxxxxxxx> 於 2021年1月29日 週五 下午1:50寫道：
>
> [loop bxxxjxxg@xxxxxxxxx] in order to reply in plain-text
> Miklos Szeredi <miklos@xxxxxxxxxx> 於 2021-01-28 16:37 寫道：
>
> On Thu, Jan 28, 2021 at 3:21 AM bingjingc <bingjingc@xxxxxxxxxxxx> wrote:
> >
> > From: BingJing Chang <bingjingc@xxxxxxxxxxxx>
> >
> > Will be used by fs parsing options
> >
> > Reviewed-by: Robbie Ko<robbieko@xxxxxxxxxxxx>
> > Reviewed-by: Chung-Chiang Cheng <cccheng@xxxxxxxxxxxx>
> > Signed-off-by: BingJing Chang <bingjingc@xxxxxxxxxxxx>
> > ---
> > fs/isofs/inode.c | 16 ++--------------
> > fs/udf/super.c | 16 ++--------------
> > include/linux/parser.h | 1 +
> > lib/parser.c | 22 ++++++++++++++++++++++
> > 4 files changed, 27 insertions(+), 28 deletions(-)
> >
> > diff --git a/fs/isofs/inode.c b/fs/isofs/inode.c
> > index 342ac19..21edc42 100644
> > --- a/fs/isofs/inode.c
> > +++ b/fs/isofs/inode.c
> > @@ -335,18 +335,6 @@ static const match_table_t tokens = {
> > {Opt_err, NULL}
> > };
> >
> > -static int isofs_match_uint(substring_t *s, unsigned int *res)
> > -{
> > - int err = -ENOMEM;
> > - char *buf = match_strdup(s);
> > -
> > - if (buf) {
> > - err = kstrtouint(buf, 10, res);
> > - kfree(buf);
> > - }
> > - return err;
> > -}
>
> I don't see how adding this function and removing it in the same
> series makes any sense.
That's true. Simple and clear is better.
I used to think that the acceptance of patch can be 3/3 or 2/3.
And I was not sure are there needs for making match_uint
as shared lib. So I made the first patch.
I simplify them. Please see the third patch, thanks!
>
> Why not make this the first patch in the series, simplifying everything?
>
> And while at it the referenced fuse implementation can also be
> converted (as a separate patch).
>
> Thanks,
> Miklos
Thanks,
BingJing
