mbox series

[nft,0/5] Fix and improve for 0021prio_0 in tests/shell

Message ID 20180925122416.15224-1-phil@nwl.cc
Headers show
Series Fix and improve for 0021prio_0 in tests/shell | expand

Message

Phil Sutter Sept. 25, 2018, 12:24 p.m. UTC 
Patch 1 contains a fix for parser_bison.y to accept 'out' as priority
name (again a keyword vs. string issue).

The remaining patches deal with flaws in the test case itself, including
the whopping 38s it took to complete on my testing VM.

Phil Sutter (5):
  parser_bison: Fix for chain prio name 'out'
  tests: shell: Fix indenting in 0021prio_0
  tests: shell: Drop one-time use variables in 0021prio_0
  tests: shell: Improve gen_chains() in 0021prio_0
  tests: shell: Improve performance of 0021prio_0

 src/parser_bison.y                      | 15 +++-
 tests/shell/testcases/chains/0021prio_0 | 91 +++++++++++--------------
 2 files changed, 51 insertions(+), 55 deletions(-)

Comments

Florian Westphal Sept. 26, 2018, 8:33 a.m. UTC | #1 
Phil Sutter <phil@nwl.cc> wrote:
> Patch 1 contains a fix for parser_bison.y to accept 'out' as priority
> name (again a keyword vs. string issue).
> 
> The remaining patches deal with flaws in the test case itself, including
> the whopping 38s it took to complete on my testing VM.

Applied, thanks Phil.