From: Eli Zaretskii <eliz@gnu.org>
To: Paul Eggert <eggert@cs.ucla.edu>
Cc: emacs-devel@gnu.org
Subject: Re: Concurrency has landed
Date: Sat, 10 Dec 2016 20:24:05 +0200 [thread overview]
Message-ID: <83a8c3fxru.fsf@gnu.org> (raw)
In-Reply-To: <83inqrfzzp.fsf@gnu.org> (message from Eli Zaretskii on Sat, 10 Dec 2016 19:36:10 +0200)
> Date: Sat, 10 Dec 2016 19:36:10 +0200
> From: Eli Zaretskii <eliz@gnu.org>
> Cc: tom@tromey.com
>
> > If no serious issues are uncovered, this will land on master by
> > tomorrow evening (UTC).
>
> Now done. Welcome to concurrency, Emacs!
Paul,
Unfortunately, this effectively reverts your changes in 644fc17, which
removed the interpreter's byte stack. The thread code saves and
restores the stack upon thread switch; since that removal gave me
trouble both during merge and while testing the results, I concluded
that the byte stack should be restored.
At the time, this conclusion made perfect sense to me. However, in
hindsight, I can no longer convince myself the removal was justified.
So if you find that the byte stack can still be removed without
breaking the concurrency feature, feel free to do so, or just tell
what should be changed in the thread-related code, and I will do that.
Sorry about this.
next prev parent reply other threads:[~2016-12-10 18:24 UTC|newest]
Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-12-09 14:45 Please test the merge of the concurrency branch Eli Zaretskii
2016-12-09 15:30 ` Robert Marshall
2016-12-09 16:26 ` Eli Zaretskii
2016-12-09 16:40 ` Paul Eggert
2016-12-09 16:54 ` Eli Zaretskii
2016-12-09 16:58 ` jpff
2016-12-09 17:06 ` Eli Zaretskii
2016-12-09 16:52 ` Robert Marshall
2016-12-09 23:09 ` Clément Pit--Claudel
2016-12-10 7:14 ` Eli Zaretskii
2016-12-10 15:05 ` Clément Pit--Claudel
2016-12-10 15:50 ` Eli Zaretskii
2016-12-09 22:33 ` David Caldwell
2016-12-10 6:50 ` Eli Zaretskii
2016-12-10 13:14 ` Alan Third
2016-12-10 14:06 ` Eli Zaretskii
2016-12-09 22:58 ` Karl Fogel
2016-12-10 7:09 ` Eli Zaretskii
2016-12-12 3:05 ` Karl Fogel
2016-12-09 23:19 ` Andrés Ramírez
2016-12-10 7:14 ` Eli Zaretskii
2016-12-10 8:09 ` Andreas Politz
2016-12-10 9:07 ` Eli Zaretskii
2016-12-10 16:05 ` Joseph Mingrone
2016-12-10 17:50 ` Eli Zaretskii
2016-12-10 16:33 ` Achim Gratz
2016-12-10 16:56 ` Filipe Silva
2016-12-10 17:56 ` Eli Zaretskii
2016-12-10 17:52 ` Eli Zaretskii
2016-12-10 17:36 ` Concurrency has landed (was: Please test the merge of the concurrency branch) Eli Zaretskii
2016-12-10 18:24 ` Eli Zaretskii [this message]
2016-12-10 18:26 ` Concurrency has landed raman
2016-12-10 18:36 ` Eli Zaretskii
2016-12-10 19:22 ` Paul Eggert
2016-12-22 4:52 ` Daniel Colascione
2016-12-22 19:23 ` Paul Eggert
2016-12-22 19:32 ` Daniel Colascione
2016-12-22 19:56 ` Paul Eggert
2016-12-24 0:52 ` Stefan Monnier
2016-12-24 1:51 ` Paul Eggert
2016-12-24 4:49 ` Stefan Monnier
2016-12-24 5:04 ` Paul Eggert
2016-12-22 19:57 ` Davis Herring
2016-12-13 12:28 ` Phillip Lord
2016-12-13 16:09 ` Eli Zaretskii
2016-12-11 2:51 ` Please test the merge of the concurrency branch Ken Raeburn
2016-12-11 3:33 ` Eli Zaretskii
2016-12-11 19:40 ` Peter Wang
2016-12-11 20:31 ` Eli Zaretskii
2016-12-11 9:11 ` Michael Albinus
2016-12-11 13:54 ` Ken Raeburn
2016-12-11 15:14 ` Tramp and concurrency (was: Please test the merge of the concurrency branch) Michael Albinus
2016-12-11 15:45 ` Please test the merge of the concurrency branch Eli Zaretskii
2016-12-11 18:30 ` Daimrod
2016-12-11 18:42 ` Eli Zaretskii
2016-12-11 19:02 ` Daimrod
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=83a8c3fxru.fsf@gnu.org \
--to=eliz@gnu.org \
--cc=eggert@cs.ucla.edu \
--cc=emacs-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.