I think this version (V4) should reflect the latest round of feedback. Please 
let me know if there are any other questions or outstanding work to finish here.

I have submitted a patch series to have a second component use this memory pool 
[1].

Thank you

[1] https://public-inbox.org/git/20180417163400.3875-2-jam...@microsoft.com/

-----Original Message-----
From: Jameson Miller 
Sent: Wednesday, April 11, 2018 2:38 PM
To: git@vger.kernel.org
Cc: gits...@pobox.com; p...@peff.net; sunsh...@sunshineco.com; 
ram...@ramsayjones.plus.com; Jameson Miller <jam...@microsoft.com>
Subject: [PATCH v4 0/3] Extract memory pool logic into reusable component

Thank you everyone for taking the time review and provide feedback on this 
patch series.

Changes from v3:

  - Based patch off of new commit, to resolve merge conflict.

  - Updated log message in 2/3 based on feedback.

  - Squashed patch from Ramsay Jones into 2/3 to fix warning from
    sparse.

  - Updated variable names in 2/3 to reflect updated usage of
    variable.

Jameson Miller (3):
  fast-import: rename mem_pool type to mp_block
  fast-import: introduce mem_pool type
  Move reusable parts of memory pool into its own file

 Makefile      |  1 +
 fast-import.c | 77 +++++++++++++----------------------------------------------
 mem-pool.c    | 55 ++++++++++++++++++++++++++++++++++++++++++
 mem-pool.h    | 34 ++++++++++++++++++++++++++
 4 files changed, 106 insertions(+), 61 deletions(-)  create mode 100644 
mem-pool.c  create mode 100644 mem-pool.h

--
2.14.3

Reply via email to