Hi,

Here is an updated version of the patch that I sent above (which was
itself an updated version of a patch that I sent to some other
sub-thread...).

In this version, I added an extra patch at the beginning to clean up
some surrounding tests in t5537 so the new ones are added both in a good
style, and in a way that is consistent with the surrounding code.

I also adopted a suggestion from Junio that introduces
'{commit,rollback}_shallow_file()' as thin wrappers around
'{commit,rollback}_lock_file()', which makes this whole thing a lot
tidier.

Thanks in advance for another look...

Taylor Blau (2):
  t5537: use test_write_lines, indented heredocs for readability
  shallow.c: use '{commit,rollback}_shallow_file'

 builtin/receive-pack.c   |  4 +-
 commit.h                 |  2 +
 fetch-pack.c             | 10 ++--
 shallow.c                | 30 ++++++++----
 t/t5537-fetch-shallow.sh | 99 +++++++++++++++++++---------------------
 5 files changed, 77 insertions(+), 68 deletions(-)

--
2.26.0.113.ge9739cdccc