From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Michael Albinus Newsgroups: gmane.emacs.devel Subject: Re: vc-state and unregistered Date: Tue, 19 Apr 2016 10:10:59 +0200 Message-ID: <87lh4arpbg.fsf@gmx.de> References: <6ok2vyzwf9.fsf@fencepost.gnu.org> <08f70cda-44be-0657-e50a-2b2c80d2c21c@yandex.ru> <87oa9dzgl0.fsf@gmx.de> <87potshczh.fsf@gmx.de> <87bn57otja.fsf_-_@gmx.de> <515b3451-820d-ad50-8c98-d0330a0f4266@yandex.ru> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: ger.gmane.org 1461053506 1838 80.91.229.3 (19 Apr 2016 08:11:46 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Tue, 19 Apr 2016 08:11:46 +0000 (UTC) Cc: emacs-devel@gnu.org To: Dmitry Gutov Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Tue Apr 19 10:11:37 2016 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 1asQky-0004Sr-4g for ged-emacs-devel@m.gmane.org; Tue, 19 Apr 2016 10:11:36 +0200 Original-Received: from localhost ([::1]:53557 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1asQkx-00011v-DH for ged-emacs-devel@m.gmane.org; Tue, 19 Apr 2016 04:11:35 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:58556) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1asQkU-0000cw-V1 for emacs-devel@gnu.org; Tue, 19 Apr 2016 04:11:12 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1asQkP-0002Sb-VN for emacs-devel@gnu.org; Tue, 19 Apr 2016 04:11:06 -0400 Original-Received: from mout.gmx.net ([212.227.15.19]:53130) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1asQkP-0002SU-M7 for emacs-devel@gnu.org; Tue, 19 Apr 2016 04:11:01 -0400 Original-Received: from detlef.gmx.de ([87.146.61.71]) by mail.gmx.com (mrgmx003) with ESMTPSA (Nemesis) id 0MSduu-1bHd8y0vZ6-00RcrH; Tue, 19 Apr 2016 10:11:00 +0200 In-Reply-To: <515b3451-820d-ad50-8c98-d0330a0f4266@yandex.ru> (Dmitry Gutov's message of "Tue, 19 Apr 2016 00:11:23 +0300") User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1.50 (gnu/linux) X-Provags-ID: V03:K0:lObb8uISNrC6XlQgkt75gblHV7Sh4mKVP/bmGRDVW8OVhqiCodO jj6HMiHvBZesiOZFFEEEdi2xKl1yMQUGOibc1xavNep5xqNXSq4ithIeZASDrhL8gZhqx+i 4EnlGIv7Y6XZYQtkxDWtBz4LUgIUNW8VT2BcG6k1/i6k3y0lCOohHR9g/vNwAM0DyAK9m+d xqiQnWGdBa/njn8CiEdxQ== X-UI-Out-Filterresults: notjunk:1;V01:K0:r2R0QHj76js=:PZ9CAiIDvQhwQeVyHn2EC7 on6Ssx5taIak38HxBjdCyc6V6ELSheXs9lh2Z5eV8SLeG/a4gmrJdKZ/j1ar99TjQsFjKkDa+ 3p69fVOmZTdqaiyqw2N0fKHg9KKJ7w7H8JXSR/9WRWvozyh2bp59o8LLTXHZfI6J7da8ETjLj dSLpQECfjqdCmdM6bGJW3ZMR7U+ya3maKn1uJ579kIlztr0S2VbPHHBmcYzDz17njwhzXiYMd huX3sBB6vS9EYhW8m3RUW0B92nVqYc0Crmuikr2voWUYmnJFQN/gBpfIW/kawWl2q/CdF8WVD oUPPfY0wYVOCd2ZB/Ad6+Zhv5ZtmpnpgopEorEkEx/msnjKA70/DZe6aYZp0Bub7ZJQcPpwNX eCXqoMrRHQUWKNj0utLTrqatcrdexi7vN3KD3YY5ltJN86keXjAUHiuAXS0bCnGK3qrbpOi8K zoH+P0P9XYQKjSPmYQvEpZvpB/4prnypU7e+VJrknVPOXQkFmEEY2q0vcpL+QVy6gZWtcG/9r hcZE5i6Y/o2ANm1cTxB8YcrP5T42T0n8ZqhSXnkSKbnhfHoObXrc2pbcabQQuOoxagA0Ulfvr K5B9JnqbaDIPoqpZwx4/prMhquhlB7Ki3qfigHzQLX5iOdXjCM6EGMXshNQDdkf8JwwTt9jOq Ils4u99k73wvvhF7goYLHDKf9EHOVuxw1rzIDco9Q0bmrbD020StJAjz8s8CzypgVr+45tKsW uj61Xsyd1iozLyKDmg2Bg5wMzh+BjI489g7hkIdf6THiytFcUiowjoUFShJ71V6SraNvCz9z X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 212.227.15.19 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:203086 Archived-At: Dmitry Gutov writes: > Hi Michael, Hi Dmitry, >> I've checked a little bit more. If we use nil instead of unregistered as >> result from vc-state, we cannot use file properties for this function >> anymore. nil is meant as "no property set". This would be a regression. > > Regression how? We already do exactly that: return `nil' in vc-state > to mean "unregistered or unknown", and have been doing this for a > while. I don't mean this. The problem is how vc caches values. It uses file properties, and in case `vc-file-getprop' returns nil, it assumes the property is not cached. Consequently, we can never cache the value nil for the file property `vc-state'. That is a regression, compared with caching the value `unregistered'. Best regards, Michael.