Re: remove reset_shared()
От | Julien Rouhaud |
---|---|
Тема | Re: remove reset_shared() |
Дата | |
Msg-id | 20220330011942.fpouca2laocok7ng@jrouhaud обсуждение исходный текст |
Ответ на | remove reset_shared() (Nathan Bossart <nathandbossart@gmail.com>) |
Ответы |
Re: remove reset_shared()
|
Список | pgsql-hackers |
Hi, On Tue, Mar 29, 2022 at 03:17:02PM -0700, Nathan Bossart wrote: > Hi hackers, > > Is there any reason to keep reset_shared() around anymore? It is now just > a wrapper function for CreateSharedMemoryAndSemaphores(), and AFAICT the > information in the comments is already covered by comments in the shared > memory code. I think it's arguable that the name of the function makes it > clear that it might recreate the shared memory, but if that is a concern, > perhaps we could rename the function to something like > CreateOrRecreateSharedMemoryAndSemaphores(). > > I've attached a patch that simply removes this wrapper function. This is > admittedly just nitpicking, so I don't intend to carry this patch further > if anyone is opposed. I'm +0.5 for it, it doesn't bring much and makes things a bit harder to understand, as you need to go through an extra function.
В списке pgsql-hackers по дате отправления: