Nguyễn Thái Ngọc Duy <pclo...@gmail.com> writes: > The struct declaration and implementation macros are moved to > commit-slab-hdr.h and commit-slab-impl.h respectively. This right now > is not needed for current users but if we share a commit-slab for > multiple files, we need something better than the current structure.
"something better" needs to be replaced with something better for it to be worth being there. In other words, say why you want to avoid including the part you moved to *-impl.h everywhere. I also think you want to rename s/-hdr.h/-decl.h/ or something to avoid sounding silly by repeating "header" twice in the name. > Signed-off-by: Nguyễn Thái Ngọc Duy <pclo...@gmail.com> > --- > commit-slab-hdr.h | 30 ++++++++++++ > commit-slab-impl.h | 91 +++++++++++++++++++++++++++++++++++ > commit-slab.h | 115 +++------------------------------------------ > 3 files changed, 127 insertions(+), 109 deletions(-) > create mode 100644 commit-slab-hdr.h > create mode 100644 commit-slab-impl.h "git blame" tells me that the new two files consist mostly of the material from commit-slab.h moved verbatim, which is assuring ;-).
