From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED!not-for-mail From: Stefan Monnier Newsgroups: gmane.emacs.devel Subject: Re: [Emacs-diffs] master b88e7c8: Make transpose-regions interactive (Bug#30343) Date: Thu, 22 Mar 2018 09:39:35 -0400 Message-ID: References: <20180311105533.30002.78782@vcs0.savannah.gnu.org> <20180311105534.3DAFD23CF3@vcs0.savannah.gnu.org> <83woyiscns.fsf@gnu.org> <87po44jb7w.fsf@red-bean.com> <20180311105533.30002.78782@vcs0.savannah.gnu.org> <20180311105534.3DAFD23CF3@vcs0.savannah.gnu.org> <83woyiscns.fsf@gnu.org> <87a7v2zb27.fsf@red-bean.com> <838tallwpg.fsf@gnu.org> <87a7v0brtb.fsf@red-bean.com> NNTP-Posting-Host: blaine.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: blaine.gmane.org 1521725905 25095 195.159.176.226 (22 Mar 2018 13:38:25 GMT) X-Complaints-To: usenet@blaine.gmane.org NNTP-Posting-Date: Thu, 22 Mar 2018 13:38:25 +0000 (UTC) User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux) To: emacs-devel@gnu.org Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Thu Mar 22 14:38:20 2018 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by blaine.gmane.org with esmtp (Exim 4.84_2) (envelope-from ) id 1ez0Q8-0006R6-QQ for ged-emacs-devel@m.gmane.org; Thu, 22 Mar 2018 14:38:20 +0100 Original-Received: from localhost ([::1]:60858 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ez0SA-0000gP-Dw for ged-emacs-devel@m.gmane.org; Thu, 22 Mar 2018 09:40:26 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:45174) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ez0RY-0000gB-Oh for emacs-devel@gnu.org; Thu, 22 Mar 2018 09:39:49 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ez0RS-0002Dj-UI for emacs-devel@gnu.org; Thu, 22 Mar 2018 09:39:48 -0400 Original-Received: from [195.159.176.226] (port=43590 helo=blaine.gmane.org) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1ez0RS-0002CM-Mo for emacs-devel@gnu.org; Thu, 22 Mar 2018 09:39:42 -0400 Original-Received: from list by blaine.gmane.org with local (Exim 4.84_2) (envelope-from ) id 1ez0PN-0005ff-5j for emacs-devel@gnu.org; Thu, 22 Mar 2018 14:37:33 +0100 X-Injected-Via-Gmane: http://gmane.org/ Original-Lines: 28 Original-X-Complaints-To: usenet@blaine.gmane.org Cancel-Lock: sha1:dhGbxRQQLT0sVDmEThhdusi7myY= X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] [fuzzy] X-Received-From: 195.159.176.226 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.21 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" Xref: news.gmane.org gmane.emacs.devel:223915 Archived-At: I haven't really participated so far since it's definitely a bikeshed's color kind of discussion, but since I'm to blame for it, I'll try and explain why I brought it up: I said I "dislike seeing" such code, without any further explanation because indeed, there's no strong technical reason behind it (I could come up with various technical reasons, but they'd just be rationalizations). I guess it just reflects my "strong types" background, where I like my code to be sanity checked by a static analyzer. For Elisp we don't have any strong static analyzer available, so we use the next best thing: our byte-compiler's warnings. With "Elisp embedded in C strings", we don't have any such tool support, so it makes me feel "unprotected" (we can't even be sure that parentheses are balanced without run-time testing). I didn't bring it up with the aim to try and change that specific chunk of code, but rather to get the opinion of others, and probably also to try and influence future code. >> Yes, they are all weak reasons, but so are the proposed reasons in >> favor of the change. > Yes, agreed on both counts. Clearly. Stefan