diff options
author | Jan Kanis <jan.code@jankanis.nl> | 2012-12-22 18:38:28 +0100 |
---|---|---|
committer | Jan Kanis <jan.code@jankanis.nl> | 2012-12-22 18:38:28 +0100 |
commit | 8a446f43ff184fad6f7aeca05fca51bb3aac2d91 (patch) | |
tree | bfa5dbe4803b4c74a7889f0a305b77f4c5b49c7f /builtin.cpp | |
parent | 30392bf66a79b91ad2fa10b369f151540c40c09e (diff) |
include fixes and suggestions from code review
Diffstat (limited to 'builtin.cpp')
-rw-r--r-- | builtin.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/builtin.cpp b/builtin.cpp index 30421de3..7df552e2 100644 --- a/builtin.cpp +++ b/builtin.cpp @@ -1010,12 +1010,12 @@ static int builtin_emit(parser_t &parser, wchar_t **argv) } - wcstring_list_t args; - wchar_t *eventname = argv[woptind]; - for (woptind++; woptind < argc; woptind++) - { - args.push_back(argv[woptind]); + if(!argv[woptind]) { + append_format(stderr_buffer, L"%ls: expected event name\n", argv[0]); + return STATUS_BUILTIN_ERROR; } + wchar_t *eventname = argv[woptind]; + wcstring_list_t args(argv + woptind + 1, argv + argc); event_fire_generic(eventname, &args); return STATUS_BUILTIN_OK; |