v1 patch failed on simple
./scripts/run.py  -e "/asdf"

Problem was in:
[PATCH 4/6] command line: use loader_parse_cmdline
core/commands:cc
assert(cmdline == __loader_argv[0]);

v2 updates only this. The string returned by strdup is now stored into
static variable, so it can be easly free-d later.

Justin

Justin Cinkelj (6):
  libc: add stresep
  libc: fix of-by-one error in stresep
  command line: add loader_parse_cmdline with tests
  command line: use loader_parse_cmdline
  command line: add tests for empty commandline string parsing
  command line: check for empty cmdline case after parsing cmdline

 Makefile                |   1 +
 arch/x64/boot.S         |   4 +-
 core/commands.cc        | 139 +++++++++++++++++++++----
 include/api/string.h    |   1 +
 include/osv/commands.hh |   6 +-
 libc/string/stresep.c   |  82 +++++++++++++++
 loader.cc               |  55 ++++------
 tests/tst-commands.cc   | 268 ++++++++++++++++++++++++++++++++++++++++++++++++
 8 files changed, 498 insertions(+), 58 deletions(-)
 create mode 100644 libc/string/stresep.c

-- 
2.9.4

-- 
You received this message because you are subscribed to the Google Groups "OSv 
Development" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to osv-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to