mbox series

[v9,00/21] Convert "git stash" to C builtin

Message ID cover.1537913094.git.ungureanupaulsebastian@gmail.com (mailing list archive)
Headers show
Series Convert "git stash" to C builtin | expand

Message

Paul-Sebastian Ungureanu Sept. 25, 2018, 10:33 p.m. UTC
Hello,

This is a new iteration of `git stash`, based on the last review I got.
This new iteration brings mostly code styling fix issues in order to make
the code more readable. There is also a new patch "strbuf.c: add
`strbuf_join_argv()`". By making some small changes, the code is now a
little bit closer to be used as API.

Joel Teichroeb (5):
  stash: improve option parsing test coverage
  stash: convert apply to builtin
  stash: convert drop and clear to builtin
  stash: convert branch to builtin
  stash: convert pop to builtin

Paul-Sebastian Ungureanu (16):
  sha1-name.c: add `get_oidf()` which acts like `get_oid()`
  strbuf.c: add `strbuf_join_argv()`
  stash: update test cases conform to coding guidelines
  stash: rename test cases to be more descriptive
  stash: add tests for `git stash show` config
  stash: convert list to builtin
  stash: convert show to builtin
  stash: mention options in `show` synopsis.
  stash: convert store to builtin
  stash: convert create to builtin
  stash: convert push to builtin
  stash: make push -q quiet
  stash: convert save to builtin
  stash: convert `stash--helper.c` into `stash.c`
  stash: optimize `get_untracked_files()` and `check_changes()`
  stash: replace all `write-tree` child processes with API calls

 Documentation/git-stash.txt  |    4 +-
 Makefile                     |    2 +-
 builtin.h                    |    1 +
 builtin/stash.c              | 1595 ++++++++++++++++++++++++++++++++++
 cache.h                      |    1 +
 git-stash.sh                 |  752 ----------------
 git.c                        |    1 +
 sha1-name.c                  |   19 +
 strbuf.c                     |   15 +
 strbuf.h                     |    7 +
 t/t3903-stash.sh             |  192 ++--
 t/t3907-stash-show-config.sh |   83 ++
 12 files changed, 1851 insertions(+), 821 deletions(-)
 create mode 100644 builtin/stash.c
 delete mode 100755 git-stash.sh
 create mode 100755 t/t3907-stash-show-config.sh

Comments

Junio C Hamano Sept. 26, 2018, 6:37 p.m. UTC | #1
Paul-Sebastian Ungureanu <ungureanupaulsebastian@gmail.com> writes:

> Hello,
>
> This is a new iteration of `git stash`, based on the last review I got.
> This new iteration brings mostly code styling fix issues in order to make
> the code more readable. There is also a new patch "strbuf.c: add
> `strbuf_join_argv()`". By making some small changes, the code is now a
> little bit closer to be used as API.
>
> Joel Teichroeb (5):
>   stash: improve option parsing test coverage
>   stash: convert apply to builtin
>   stash: convert drop and clear to builtin
>   stash: convert branch to builtin
>   stash: convert pop to builtin
>
> Paul-Sebastian Ungureanu (16):
>   sha1-name.c: add `get_oidf()` which acts like `get_oid()`
>   strbuf.c: add `strbuf_join_argv()`
>   stash: update test cases conform to coding guidelines
>   stash: rename test cases to be more descriptive
>   stash: add tests for `git stash show` config
>   stash: convert list to builtin
>   stash: convert show to builtin
>   stash: mention options in `show` synopsis.
>   stash: convert store to builtin
>   stash: convert create to builtin
>   stash: convert push to builtin
>   stash: make push -q quiet
>   stash: convert save to builtin
>   stash: convert `stash--helper.c` into `stash.c`
>   stash: optimize `get_untracked_files()` and `check_changes()`
>   stash: replace all `write-tree` child processes with API calls

Here is how these appear in my MUA.  I am guessing that 03/21 is the
right copy and 02/21 that appear after 21/21 is a duplicat that I
should not even look at (the same for 4 other ones with [GSoC]
label)?

   [  53: PSU] [PATCH v9 00/21] Convert "git stash" to C builtin
    [  57: PSU] [PATCH v9 01/21] sha1-name.c: add `get_oidf()` which acts like `get_oid()`
    [  56: PSU] [PATCH v9 02/21] strbuf.c: add `strbuf_join_argv()`
    [  68: PSU] [PATCH v9 03/21] stash: improve option parsing test coverage
    [ 378: PSU] [PATCH v9 04/21] stash: update test cases conform to coding guidelines
    [  78: PSU] [PATCH v9 05/21] stash: rename test cases to be more descriptive
    [ 102: PSU] [PATCH v9 06/21] stash: add tests for `git stash show` config
    [ 632: PSU] [PATCH v9 07/21] stash: convert apply to builtin
    [ 199: PSU] [PATCH v9 08/21] stash: convert drop and clear to builtin
    [ 132: PSU] [PATCH v9 09/21] stash: convert branch to builtin
    [ 151: PSU] [PATCH v9 10/21] stash: convert pop to builtin
    [ 100: PSU] [PATCH v9 11/21] stash: convert list to builtin
    [ 296: PSU] [PATCH v9 12/21] stash: convert show to builtin
    [  56: PSU] [PATCH v9 13/21] stash: mention options in `show` synopsis.
    [ 165: PSU] [PATCH v9 14/21] stash: convert store to builtin
    [ 513: PSU] [PATCH v9 15/21] stash: convert create to builtin
    [ 339: PSU] [PATCH v9 16/21] stash: convert push to builtin
    [ 195: PSU] [PATCH v9 17/21] stash: make push -q quiet
    [ 430: PSU] [PATCH v9 18/21] stash: convert save to builtin
    [ 575: PSU] [PATCH v9 19/21] stash: convert `stash--helper.c` into `stash.c`
    [ 156: PSU] [PATCH v9 20/21] stash: optimize `get_untracked_files()` and `check_changes()`
    [ 130: PSU] [PATCH v9 21/21] stash: replace all `write-tree` child processes with API calls
   [  68: PSU] [GSoC][PATCH v9 02/21] stash: improve option parsing test coverage
   [ 378: PSU] [GSoC][PATCH v9 03/21] stash: update test cases conform to coding guidelines
   [  78: PSU] [GSoC][PATCH v9 04/21] stash: rename test cases to be more descriptive
   [ 102: PSU] [GSoC][PATCH v9 05/21] stash: add tests for `git stash show` config
   [  55: PSU] [GSoC][PATCH v9 06/21] strbuf.c: add `strbuf_join_argv()`

Thanks.

>
>  Documentation/git-stash.txt  |    4 +-
>  Makefile                     |    2 +-
>  builtin.h                    |    1 +
>  builtin/stash.c              | 1595 ++++++++++++++++++++++++++++++++++
>  cache.h                      |    1 +
>  git-stash.sh                 |  752 ----------------
>  git.c                        |    1 +
>  sha1-name.c                  |   19 +
>  strbuf.c                     |   15 +
>  strbuf.h                     |    7 +
>  t/t3903-stash.sh             |  192 ++--
>  t/t3907-stash-show-config.sh |   83 ++
>  12 files changed, 1851 insertions(+), 821 deletions(-)
>  create mode 100644 builtin/stash.c
>  delete mode 100755 git-stash.sh
>  create mode 100755 t/t3907-stash-show-config.sh