Re: [PATCH 0/3] Make support for multi-value configuration options more consistent

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2024-04-02 02:13, Dragan Simic wrote:
This series adds support for optional placing of additional whitespace
characters around the commas in the multi-value, comma-separated values
for the "diff.dirstat" and "diff.wsErrorHighlight" configuration options,
and the "--dirstat" and "--ws-error-highlight" command-line options.

This makes the way multi-(sub)value configuration options can be specified more consistent, because exactly the same additional whitespace characters
are already allowed for some of the configuration options.  It may also
make the configuration files more readable to some users.

Meh, somehow I managed to forget to include the brief summary for the
"what's cooking".  Here's what might be used for that purpose:

* Allow additional whitespace around the commas in "diff.dirstat=<multi-value>", "diff.wsErrorHighlight=<multi-value>", "git diff --dirstat=<multi-value>"
   and "git diff --ws-error-highlight=<multi-value>".

Dragan Simic (3):
diff: support additional whitespace in diff.{dirstat,wsErrorHighlight}
  t4015: whitespace in diff.wsErrorHighlight and --ws-error-highlight
  t4047: whitespace in diff.dirstat and --dirstat

 builtin/gc.c                |  4 ++--
 diff.c                      | 14 ++++++++++----
 notes.c                     |  2 +-
 refs/packed-backend.c       |  2 +-
 string-list.c               | 27 +++++++++++++++++++++------
 string-list.h               | 10 ++++++----
 t/helper/test-hashmap.c     |  2 +-
 t/helper/test-json-writer.c |  2 +-
 t/helper/test-oidmap.c      |  2 +-
 t/helper/test-string-list.c |  4 ++--
 t/t4015-diff-whitespace.sh  | 36 ++++++++++++++++++++++++++++++++++--
 t/t4047-diff-dirstat.sh     | 33 ++++++++++++++++++++++++++++++---
 12 files changed, 110 insertions(+), 28 deletions(-)




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux