[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 06/10] add most lisp-level features
From: |
Daniel Colascione |
Subject: |
Re: [PATCH 06/10] add most lisp-level features |
Date: |
Thu, 09 Aug 2012 18:45:35 -0700 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:12.0) Gecko/20120428 Thunderbird/12.0.1 |
On 8/9/2012 12:41 PM, Tom Tromey wrote:
> This adds most of the thread features visible to emacs lisp.
>
> I roughly followed the Bordeaux threads API:
>
> http://trac.common-lisp.net/bordeaux-threads/wiki/ApiDocumentation
>
> ... but not identically. In particular I chose not to implement
> interrupt-thread or destroy-thread, but instead a thread-signalling
> approach.
Thank you. Functions like that lead to nothing but trouble.
> I'm still undecided about *default-special-bindings* (which I did not
> implement). I think it would be more emacs-like to capture the let
> bindings at make-thread time, but IIRC Stefan didn't like this idea
> the first time around.
I'm with Stefan here. Capturing the bindings at thread-creation time makes the
binding in effect for a thread worker hard to reason about. Say Gnus binds *foo*
and calls into bar-lib, which internally uses threads. bar-lib's thread worker
has no idea *foo* is bound, and if a non-Gnus caller uses bar-lib, *foo* won't
be bound. I feel like this approach would lead to hard-to-find bugs.
signature.asc
Description: OpenPGP digital signature
- [PATCH 06/10] add most lisp-level features, Tom Tromey, 2012/08/09
- Re: [PATCH 06/10] add most lisp-level features,
Daniel Colascione <=
- Re: [PATCH 06/10] add most lisp-level features, Tom Tromey, 2012/08/13
- Re: [PATCH 06/10] add most lisp-level features, Stefan Monnier, 2012/08/13
- Re: [PATCH 06/10] add most lisp-level features, Tom Tromey, 2012/08/13
- Re: [PATCH 06/10] add most lisp-level features, Stefan Monnier, 2012/08/13
- Re: [PATCH 06/10] add most lisp-level features, Tom Tromey, 2012/08/14
- Re: [PATCH 06/10] add most lisp-level features, Stefan Monnier, 2012/08/14