On December 17, 2016 3:56:19 AM PST, Philip Oakley <philipoakley@xxxxxxx> wrote: >From: "Jacob Keller" <jacob.e.keller@xxxxxxxxx> >> From: Jacob Keller <jacob.keller@xxxxxxxxx> >> >> Commit c8ba16391655 ("parse-options: add OPT_STRING_LIST helper", >> 2011-06-09) added the OPT_STRING_LIST as a way to accumulate a >repeated >> list of strings. However, this was not documented in the >> api-parse-options documentation. Add documentation now so that future >> developers may learn of its existence. >> >> Signed-off-by: Jacob Keller <jacob.keller@xxxxxxxxx> >> --- >> Documentation/technical/api-parse-options.txt | 5 +++++ >> 1 file changed, 5 insertions(+) >> >> diff --git a/Documentation/technical/api-parse-options.txt >> b/Documentation/technical/api-parse-options.txt >> index 27bd701c0d68..92791740aa64 100644 >> --- a/Documentation/technical/api-parse-options.txt >> +++ b/Documentation/technical/api-parse-options.txt >> @@ -168,6 +168,11 @@ There are some macros to easily define options: >> Introduce an option with string argument. >> The string argument is put into `str_var`. >> >> +`OPT_STRING_LIST(short long, &list, arg_str, description)`:: > >should there be an extra comma between 'short long' in a similar manner >to >the OPT_INTEGER argument list below? > > You are indeed correct sir. I will fix this up. Thanks, Jake