Re: [PATCH v2 00/28] Builtin FSMonitor Feature

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

 



Hi Junio,

On Thu, 27 May 2021, Junio C Hamano wrote:

> These new global symbols are introduced by the series, but never
> used outside the file they are added to:
>
> fsmonitor-ipc.o        - fsmonitor_ipc__get_path
> fsmonitor-ipc.o        - fsmonitor_ipc__get_state
> fsmonitor-ipc.o        - fsmonitor_ipc__send_command
>
> Perhaps make them file-scope static?

Good idea!

By the way, GitGitGadget keeps getting confused by the fact that one of
Peff's patches looks very similar to the tip commit of this here patch
series, and mislabels Jeff's PR as being closed.

Would you terribly mind picking up v2 some time soon so that I do not have
to click "Reopen" on Jeff's PR all the time (I had to reopen it a couple
times already: https://github.com/gitgitgadget/git/pull/923, and I do lack
the time to teach GitGitGadget new tricks to avoid this mislabeling).

Thank you,
Dscho




[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