Re: [PATCH v2 27/27] fsmonitor-ll.h: split this header out of fsmonitor.h

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, May 15, 2023 at 5:17 PM Jonathan Tan <jonathantanmy@xxxxxxxxxx> wrote:
>
> "Elijah Newren via GitGitGadget" <gitgitgadget@xxxxxxxxx> writes:
> > From: Elijah Newren <newren@xxxxxxxxx>
> >
> > This creates a new fsmonitor-ll.h with most of the functions from
> > fsmonitor.h, though it leaves three inline functions where they were.
> > Two-thirds of the files that previously included fsmonitor.h did not
> > need those three inline functions or the six extra includes those inline
> > functions required, so this allows them to only include the lower level
> > header.
> >
> > Diff best viewed with `--color-moved`.
> >
> > Signed-off-by: Elijah Newren <newren@xxxxxxxxx>
>
> Looks good. Thanks for this significant effort.

Thanks for the review!




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux