On Mon, Feb 07, 2022 at 03:53:13PM +0100, Florian Westphal wrote:
> Pablo Neira Ayuso <pablo@xxxxxxxxxxxxx> wrote:
> > On Mon, Feb 07, 2022 at 02:29:15PM +0100, Florian Westphal wrote:
> > > Florian Westphal <fw@xxxxxxxxx> wrote:
> > > > flow statement has no export, its shown as:
> > > > ".. }, "flow add @ft" ] } }"
> > > >
> > > > With this patch:
> > > >
> > > > ".. }, {"flow": {"op": "add", "flowtable": "@ft"}}]}}"
> > >
> > > This is based on the 'set' statement. If you prefer the @ to
> > > be removed let me know.
> >
> > Then, it is consistent with the existing syntax. So either we consider
> > deprecating the @ on the 'set' statement (while retaining backward
> > compatibility) or flowtable also includes it as in your patch.
>
> Ok, then lets keep it as-is; the @ might help humans to find a
> set/flowtable name more quickly this way.
OK.
