* Re: master 506270f: Don't compare arguments that can be nil [not found] ` <20171216004037.6240920856@vcs0.savannah.gnu.org> @ 2017-12-16 21:08 ` Glenn Morris 2017-12-16 23:13 ` Óscar Fuentes 0 siblings, 1 reply; 4+ messages in thread From: Glenn Morris @ 2017-12-16 21:08 UTC (permalink / raw) To: emacs-devel; +Cc: Oscar Fuentes Oscar Fuentes wrote: > branch: master > commit 506270f9c80bf9bd7dad35a2f0aa6f477da6490b [...] > * lisp/ses.el (ses--advice-copy-region-as-kill): avoid comparison > unless `beg' and `end' are non-nil. This is https://debbugs.gnu.org/28039, which seems like something to fix in the emacs-26 branch. ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: master 506270f: Don't compare arguments that can be nil 2017-12-16 21:08 ` master 506270f: Don't compare arguments that can be nil Glenn Morris @ 2017-12-16 23:13 ` Óscar Fuentes 2017-12-17 3:37 ` Eli Zaretskii 0 siblings, 1 reply; 4+ messages in thread From: Óscar Fuentes @ 2017-12-16 23:13 UTC (permalink / raw) To: Glenn Morris; +Cc: emacs-devel Glenn Morris <rgm@gnu.org> writes: > Oscar Fuentes wrote: > >> branch: master >> commit 506270f9c80bf9bd7dad35a2f0aa6f477da6490b > [...] >> * lisp/ses.el (ses--advice-copy-region-as-kill): avoid comparison >> unless `beg' and `end' are non-nil. > > This is https://debbugs.gnu.org/28039, which seems like something to > fix in the emacs-26 branch. Thanks. AFAIK emacs-26 is open only for documentation and regressions. This bug existed since the beginning of SES. OTOH, it looks quite safe. I have no problem about backporting it to emacs-26. Eli? ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: master 506270f: Don't compare arguments that can be nil 2017-12-16 23:13 ` Óscar Fuentes @ 2017-12-17 3:37 ` Eli Zaretskii 2017-12-18 0:39 ` Dmitry Gutov 0 siblings, 1 reply; 4+ messages in thread From: Eli Zaretskii @ 2017-12-17 3:37 UTC (permalink / raw) To: Óscar Fuentes; +Cc: rgm, emacs-devel > From: ofv@wanadoo.es (Óscar Fuentes) > Date: Sun, 17 Dec 2017 00:13:57 +0100 > Cc: emacs-devel@gnu.org > > > This is https://debbugs.gnu.org/28039, which seems like something to > > fix in the emacs-26 branch. > > Thanks. > > AFAIK emacs-26 is open only for documentation and regressions. I wish that were the case, but given the fact we wait for the widen-less stuff to be merged, we are a far cry from there. > This bug existed since the beginning of SES. OTOH, it looks quite > safe. > > I have no problem about backporting it to emacs-26. Eli? Please do, and thanks. ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: master 506270f: Don't compare arguments that can be nil 2017-12-17 3:37 ` Eli Zaretskii @ 2017-12-18 0:39 ` Dmitry Gutov 0 siblings, 0 replies; 4+ messages in thread From: Dmitry Gutov @ 2017-12-18 0:39 UTC (permalink / raw) To: Eli Zaretskii, Óscar Fuentes; +Cc: rgm, emacs-devel On 12/17/17 5:37 AM, Eli Zaretskii wrote: >> AFAIK emacs-26 is open only for documentation and regressions. > > I wish that were the case, but given the fact we wait for the > widen-less stuff to be merged, we are a far cry from there. FWIW, the branch is ready as far as I'm concerned. Waiting only for either of the maintainers' approval. Of course, feedback from Christoph will be welcome too, as well as any feedback on the manual edits, from everybody. ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-12-18 0:39 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- [not found] <20171216004036.16935.43030@vcs0.savannah.gnu.org> [not found] ` <20171216004037.6240920856@vcs0.savannah.gnu.org> 2017-12-16 21:08 ` master 506270f: Don't compare arguments that can be nil Glenn Morris 2017-12-16 23:13 ` Óscar Fuentes 2017-12-17 3:37 ` Eli Zaretskii 2017-12-18 0:39 ` Dmitry Gutov
Code repositories for project(s) associated with this public inbox https://git.savannah.gnu.org/cgit/emacs.git This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).