From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED!not-for-mail From: Michael Heerdegen Newsgroups: gmane.emacs.devel Subject: Re: Unnecessary variable shr-start Date: Fri, 02 Dec 2016 23:26:10 +0100 Message-ID: <87pola2cjh.fsf@web.de> References: <87zikeji0s.fsf@topbug.net> NNTP-Posting-Host: blaine.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: blaine.gmane.org 1480717618 1234 195.159.176.226 (2 Dec 2016 22:26:58 GMT) X-Complaints-To: usenet@blaine.gmane.org NNTP-Posting-Date: Fri, 2 Dec 2016 22:26:58 +0000 (UTC) User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1.50 (gnu/linux) Cc: emacs-devel@gnu.org To: Hong Xu Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Fri Dec 02 23:26:54 2016 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 1cCwI9-0007qb-Sc for ged-emacs-devel@m.gmane.org; Fri, 02 Dec 2016 23:26:53 +0100 Original-Received: from localhost ([::1]:36796 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1cCwID-0000cy-Sd for ged-emacs-devel@m.gmane.org; Fri, 02 Dec 2016 17:26:57 -0500 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:36495) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1cCwHg-0000cr-0o for emacs-devel@gnu.org; Fri, 02 Dec 2016 17:26:24 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1cCwHc-0006Fi-Tf for emacs-devel@gnu.org; Fri, 02 Dec 2016 17:26:24 -0500 Original-Received: from mout.web.de ([212.227.15.4]:59493) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1cCwHc-0006De-KQ for emacs-devel@gnu.org; Fri, 02 Dec 2016 17:26:20 -0500 Original-Received: from drachen.dragon ([90.186.0.176]) by smtp.web.de (mrweb004 [213.165.67.108]) with ESMTPSA (Nemesis) id 0M3gLx-1cU3Yl0Pp3-00rIRQ; Fri, 02 Dec 2016 23:26:14 +0100 In-Reply-To: <87zikeji0s.fsf@topbug.net> (Hong Xu's message of "Fri, 02 Dec 2016 10:35:47 -0800") X-Provags-ID: V03:K0:SSM0Gb9Wymrd2yWZ8W1njMzuv33FYhH+Udpw6PEzjkDVDKxCkW+ 62SdcXTt7/wJhtZ7iL5fyUfHf9oo5TL52D/ZfTWxkRuQXLAiG/zHAfq/TLuLUgP2L8eT2J3 yPBPdTQ3MnpZidRRIa+aD+U7ZFmiftbEhru2yPIsTWVVXHQ0TTMzxVupLJkxpPuUH2xvSii ntHzuySwCPvT6AUNkxruA== X-UI-Out-Filterresults: notjunk:1;V01:K0:g1sOkBhvVG0=:Q3By7OUxxHo4VHgD/44l4I 34S0PiC1kgqEuwxqEI7a2Qc5rRXzWIb44XS1NNdFaSrlhiWoluI8eNhdVWsBO4aotbChIMVQI zgGJbeXZ7djPbnV6Pinmq0UQXPL9WsvV5k1tdHtVY4FBDMbRkex456ojmDtnQkQCIkOcBQ30o M5T0lXl15rri0Le5U5r6VHpfMJJUC1HmjvRBvpE9z2D5W1oCZEE6fPWr6J9GdvRcV+nS/n/U/ pVr95xIe50nfkVyAs6q1hdovzxyD3s4eou2dsijnqWW+6QwOTAHOO36CLZ1SZ0yZED175zIwq EILUdZCbjOqDue6uWogCvP7E4R5FD8h21un3WMHeO8WIRLbg2Z+qEIWaX4AgxRNHCZLGORfpa LCy7cEzaq6P8bsCBRblkijxo+HJOmp2vALBA0CB+sjy03qQsGFJ7ts3ZBXpylEXJ+aGFvJjTv v4uCQ6LMfaCwgiwalK7C0/ww/Dm/mOP5b784Ca/0e/Q8cdVKzaxdItvB5Aftueq1HmwQnHzoS PXg7UCrPWrD/oENF3OdAxIgOTjMYsWmF+6200/05k/vI/9l9u6vVsRHgkgsr5ZwNKgTYM4bHC 74bpT6h9cH4qDt0kVuRkF7TybgTbQAMMgilhZNALH58rB1iVv6KekzaZpHl7g0AGFVqUiEp9O N9qcKbCnwA1YCMNji7pgy/nwsMOSzeWnAIy5F/Ovh8DIVmE0uJxrHNJ0ajILaHGI5VtnmMQMx b90Mod+NqjFtLZ6mbL5wCHAzSSRJ5tD1nOgHi4vgEKtQb8pauY9gW69i2qX/Nca5PAYZSVuT X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] [fuzzy] X-Received-From: 212.227.15.4 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:209951 Archived-At: Hong Xu writes: > In the current master branch, it seems that the variable shr-start has > never been used in any sensible way: Every time its value is referred, > it is bound to nil. Should it be considered to be removed? Are you sure it is always bound to nil when referenced in `shr-tag-a'? I would expect it can be changed by side effect in the body, and the let-binding is there to make that change temporary. Michael.