mbox series

[0/4] parse-options: simplify parse_options_concat() and parse_options_dup()

Message ID 11b82734-f61c-5e73-2d0c-22208c06d495@web.de (mailing list archive)
Headers show
Series parse-options: simplify parse_options_concat() and parse_options_dup() | expand

Message

René Scharfe Feb. 9, 2020, 3:53 p.m. UTC
Reduce code duplication.

René Scharfe (4):
  parse-options: use COPY_ARRAY in parse_options_concat()
  parse-options: factor out parse_options_count()
  parse-options: const parse_options_concat() parameters
  parse-options: simplify parse_options_dup()

 parse-options-cb.c | 42 +++++++++++++++++-------------------------
 parse-options.h    |  2 +-
 2 files changed, 18 insertions(+), 26 deletions(-)

--
2.25.0

Comments

Jeff King Feb. 10, 2020, 10:25 p.m. UTC | #1
On Sun, Feb 09, 2020 at 04:53:46PM +0100, René Scharfe wrote:

> Reduce code duplication.
> 
> René Scharfe (4):
>   parse-options: use COPY_ARRAY in parse_options_concat()
>   parse-options: factor out parse_options_count()
>   parse-options: const parse_options_concat() parameters
>   parse-options: simplify parse_options_dup()

I read all 4 and they looked good to me.

-Peff