git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCHv5 0/9] Handling empty notes

From
Johan Herland <johan@herland.net>
Date
Nov 12, 2014, 00:40 UTC
Message-ID
<1415752816-22782-1-git-send-email-johan@herland.net>
Changes v4 -> v5:
 - Patch #5: Reorder code to ease readability and fix issue with
   checking d.given after free_note_data(&d), noticed by Junio.
 - Patch #9: Fix typo in commit message, noticed by Junio.
 - Patch #9: Rename "output" files to "actual", suggested by Eric.
 - Patch #9: Use test_line_count instead of wc -l, which behaves
   differently on Macs. noticed by Michael, diagnosed by Eric, and
   fix suggested by Peff.
 - Patch #9: Use -r option to ls-tree when counting entries in notes,
   tree as this also works for trees with fanout level > 0. Noticed
   by Peff.
Have fun! :)
...Johan
Johan Herland (9):
  builtin/notes: Fix premature failure when trying to add the empty blob
  t3301: Verify that 'git notes' removes empty notes by default
  builtin/notes: Improve naming
  builtin/notes: Refactor note file path into struct note_data
  builtin/notes: Simplify early exit code in add()
  builtin/notes: Split create_note() to clarify add vs. remove logic
  builtin/notes: Add --allow-empty, to allow storing empty notes
  notes: Empty notes should be shown by 'git log'
  t3301: Modernize
 Documentation/git-notes.txt |   12 +-
 builtin/notes.c             |  258 +++++----
 notes.c                     |    3 +-
 t/t3301-notes.sh            | 1341 +++++++++++++++++++++----------------------
 4 files changed, 789 insertions(+), 825 deletions(-)
-- 
2.0.0.rc4.501.gdaf83ca
Next: Johan Herland
Message 1 of 14 in “[PATCHv5 0/9] Handling empty notes”
  1. Johan HerlandNov 12, 2014
  2. 1/9 builtin/notes: Fix premature failure when trying to add the empty blobJohan Herland, Nov 12, 2014
  3. 2/9 t3301: Verify that 'git notes' removes empty notes by defaultJohan Herland, Nov 12, 2014
  4. 3/9 builtin/notes: Improve namingJohan Herland, Nov 12, 2014
  5. 4/9 builtin/notes: Refactor note file path into struct note_dataJohan Herland, Nov 12, 2014
  6. 5/9 builtin/notes: Simplify early exit code in add()Johan Herland, Nov 12, 2014
  7. 6/9 builtin/notes: Split create_note() to clarify add vs. remove logicJohan Herland, Nov 12, 2014
  8. 7/9 builtin/notes: Add --allow-empty, to allow storing empty notesJohan Herland, Nov 12, 2014
  9. 8/9 notes: Empty notes should be shown by 'git log'Johan Herland, Nov 12, 2014
  10. 9/9 t3301: ModernizeJohan Herland, Nov 12, 2014
  11. Eric SunshineNov 12, 2014
  12. Johan HerlandNov 12, 2014
  13. Junio C HamanoNov 12, 2014
  14. Johan HerlandNov 12, 2014

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.