Gurjeet Singh <gurjeet@singh.im> writes:
> Please find attached the patch. It includes the doc changes as well.
What exactly is the point of the static state you added here? There
is no situation where that could possibly be useful, because this
code is executed at most once per process, and not at all in the
postmaster. AFAICS, that's just complication (and permanent waste
of a kilobyte of static data space) for no return.
Also, this seems to try to write the file whether or not the environment
variable was set, which wasn't the plan.
I also don't really see the point of parsing the value variable into an
int. Why not just spit it out as the original string? It's not ours to
legislate whether the kernel will take a value that's not an integer.
regards, tom lane
