[PATCH 1 of 9] atexit: add special-purpose atexit functionality to util
Jun Wu
quark at fb.com
Mon Apr 10 21:29:49 UTC 2017
Excerpts from Bryan O'Sullivan's message of 2017-04-10 13:15:56 -0700:
> On Mon, Apr 10, 2017 at 12:49 PM, Jun Wu <quark at fb.com> wrote:
> > I wonder if it's better if the global state is moved to a
> > "dispatch.request"
> > or "ui" object. That may give threaded request handlers more confidence.
>
> The new atexit code is only run during process exit. I was careful to
> ensure that I wasn't accidentally running it at the end of e.g. every web
> or chg request, but if they somehow arose, such cases would result in a
> process crash and test failure.
That makes sense. The series all look good to me.
More information about the Mercurial-devel
mailing list