Thread (1 message) 1 message, 1 author, 2021-03-10

Re: [PATCH v5 10/12] unix-stream-server: create unix domain socket under lock

From: Junio C Hamano <hidden>
Date: 2021-03-10 00:19:52

"Jeff Hostetler via GitGitGadget" [off-list ref] writes:
quoted hunk
+struct unix_stream_server_socket {
+int unix_stream_server__create(
+void unix_stream_server__free(
+int unix_stream_server__was_stolen(
I think we reserve __ in our API for names of symbols that normal
callers never have to write (both data like git_attr__true[] and
functions like cmd_bisect__helper()).

It seems that list-objects-filter.h may have introduced the
"name_space" followed by "__" followed by "name" convention,
but I am not sure if that is a desirable convention to spread
throughout our codebase.

Also "unix_stream_server" is quite a mouthful.  Perhaps abbreviate
it to uss_ or something?  I dunno if that is too short and invite
confusion with other kinds of uss.


Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help