From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: phillip.lord@russet.org.uk (Phillip Lord) Newsgroups: gmane.emacs.devel Subject: Re: ChangeLog entries Date: Sun, 09 Aug 2015 16:46:20 +0100 Message-ID: <87lhdkmpmb.fsf@russet.org.uk> References: <83vbcopkd3.fsf@gnu.org> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: ger.gmane.org 1439135197 1074 80.91.229.3 (9 Aug 2015 15:46:37 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Sun, 9 Aug 2015 15:46:37 +0000 (UTC) Cc: emacs-devel@gnu.org To: Eli Zaretskii Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Sun Aug 09 17:46:31 2015 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1ZOSnu-0001v6-U2 for ged-emacs-devel@m.gmane.org; Sun, 09 Aug 2015 17:46:31 +0200 Original-Received: from localhost ([::1]:55546 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZOSnu-00084s-BO for ged-emacs-devel@m.gmane.org; Sun, 09 Aug 2015 11:46:30 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:51979) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZOSnr-00084j-0o for emacs-devel@gnu.org; Sun, 09 Aug 2015 11:46:28 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZOSnq-0002sT-2p for emacs-devel@gnu.org; Sun, 09 Aug 2015 11:46:26 -0400 Original-Received: from cheviot22.ncl.ac.uk ([128.240.234.22]:42937) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZOSnl-0002p3-BB; Sun, 09 Aug 2015 11:46:21 -0400 Original-Received: from smtpauth-vm.ncl.ac.uk ([10.8.233.129] helo=smtpauth.ncl.ac.uk) by cheviot22.ncl.ac.uk with esmtp (Exim 4.63) (envelope-from ) id 1ZOSnk-0000Wb-FE; Sun, 09 Aug 2015 16:46:20 +0100 Original-Received: from cpc6-benw10-2-0-cust45.gate.cable.virginm.net ([92.238.179.46] helo=localhost) by smtpauth.ncl.ac.uk with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.63) (envelope-from ) id 1ZOSnk-0000LL-HY; Sun, 09 Aug 2015 16:46:20 +0100 In-Reply-To: <83vbcopkd3.fsf@gnu.org> (Eli Zaretskii's message of "Sun, 9 Aug 2015 18:11:36 +0300") User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6.x X-Received-From: 128.240.234.22 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:188643 Archived-At: Eli Zaretskii writes: > Currently, ChangeLog.2 has entries like this: > > commit 45987b34535e5ae97fa14535630e283f34af94dd > Merge: c208eef feadec3 > Author: Nicolas Petton > Date: Sat Aug 8 21:54:45 2015 +0200 > > Merge remote-tracking branch 'origin/fix/subsequence-error-with-negative-seq > > commit feadec307da148af70cf87013c99771ca4db91e4 > Author: Phillip Lord > Date: Fri Aug 7 22:12:59 2015 +0100 > > Improve error signalling for seq-subseq. > > The existing behaviour for seq-subseq is to error when indexes are too > large, but to silently ignore numbers which are too negative for lists. > String and vector handling errors in both cases. This has been > regularlised. > > Error signalling behaviour has been explicitly added to the docstring of > seq-subseq, and also to cl-subseq which largely defers to > seq-subseq (and is therefore also impacted by this change). > > Tests have been added for these exceptional cases, as well as one non > exceptional base case. > > I thought log entries from branches weren't supposed to end up in > ChangeLog, but if they are, I think we should delete these. They are both changes to trunk. feadec was a change *originally* made on a branch, 45987 is a merge commit pulling feadec onto trunk also. I wanted Nicolas to okay by change given that it was his code. Personally, I would have rebased, but a rebase vs merge discussion is unlikely to add anything substantial to the world. I'd rather feadec wasn't deleted. It's my first ChangeLog entry and I'm really rather proud of it, pathetic as that might sound. Phil