This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author neologix
Recipients flox, neologix, vstinner
Date 2011-10-23.11:28:54
SpamBayes Score 7.42475e-06
Marked as misclassified No
Message-id <CAH_1eM0zkkZWyrVreafxfiEeDo48MC0JGBHDD423XbXb4xQZBQ@mail.gmail.com>
In-reply-to <1319325270.49.0.594851522806.issue13140@psf.upfronthosting.co.za>
Content
> I would prefer to preserve the inheritance by default, and to change the daemonic attribute only if it is explicitly set to True or False.
> This way it will be backward compatible.

It may be backward compatible, but IMHO, the current behavior is
broken: while it can certainly make sense to set the server thread
daemonic, you most certainly don't want to have client threads
daemonic implicitely (since you usually don't want to terminate the
client's connections abruptly when the main thread exits).
But I must admit I don't have a strong opinion, so both solutions are OK to me.
The only thing that bothers me is this:
"""
+.. versionchanged:: 3.3
+   previously, the *daemon_threads = False* flag was ignored.
"""

You usually document new features or behavior changes: this really
looks like a bug fix (and is one actually). Something like "the
semantics of *daemon_threads* changed slighty" might be better (but
I'm no native speaker).
History
Date User Action Args
2011-10-23 11:28:55neologixsetrecipients: + neologix, vstinner, flox
2011-10-23 11:28:54neologixlinkissue13140 messages
2011-10-23 11:28:54neologixcreate