On Wed, 26 Jul 2023 at 21:02, Trevor Gross <tmgross@xxxxxxxxx> wrote:
>
> On Wed, Jul 26, 2023 at 7:48 PM Wedson Almeida Filho <wedsonaf@xxxxxxxxx> wrote:
> >
> > On Wed, 26 Jul 2023 at 18:08, Trevor Gross <tmgross@xxxxxxxxx> wrote:
> > >
> > [...]
> > > The guard syntax (available since 1.65) can make these kinds of match statements
> > > cleaner:
> >
> > This is unstable though.
> >
> > We try to stay away from unstable features unless we really need them,
> > which I feel is not the case here.
> >
>
> Let/else has been stable since 1.65 and is in pretty heavy use, the
> kernel is on 1.68.2 correct? Could you be thinking of let chains
> (multiple matches joined with `&&`/`||`)?
Oh, my bad. I got it mixed with "if let guard" (issue 51114).
And yeah, we're on 1.68.2, so this code can benefit from this simplification.
Thanks!
>
> > let Some(oci_root_dir) = data.oci_root_dir else {
> > pr_err!("missing oci_root_dir parameter!\n");
> > return Err(ENOTSUPP);
> > }
> > [...]
