From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Gregory Heytings Newsgroups: gmane.emacs.devel Subject: Re: emacs-29 b8d2ec920f: Revert "Improve last change to xfaces.c" (05ece1eb8b) Date: Mon, 12 Dec 2022 10:42:17 +0000 Message-ID: References: <167080778504.14972.16819452979975432761@vcs2.savannah.gnu.org> <20221212011625.58E8AC004B4@vcs2.savannah.gnu.org> <877cyxjqd8.fsf@yahoo.com> <87359ljq04.fsf@yahoo.com> <87tu21hr7m.fsf@yahoo.com> Mime-Version: 1.0 Content-Type: text/plain; format=flowed; charset=us-ascii Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="27177"; mail-complaints-to="usenet@ciao.gmane.io" Cc: emacs-devel@gnu.org To: Po Lu Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Mon Dec 12 11:42:49 2022 Return-path: Envelope-to: ged-emacs-devel@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1p4gGm-0006qn-UY for ged-emacs-devel@m.gmane-mx.org; Mon, 12 Dec 2022 11:42:48 +0100 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1p4gGN-0003iN-D6; Mon, 12 Dec 2022 05:42:23 -0500 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1p4gGL-0003ci-M4 for emacs-devel@gnu.org; Mon, 12 Dec 2022 05:42:22 -0500 Original-Received: from heytings.org ([95.142.160.155]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1p4gGJ-0005Vn-Rq for emacs-devel@gnu.org; Mon, 12 Dec 2022 05:42:21 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=heytings.org; s=20220101; t=1670841737; bh=4gvglRMeDzOIK3kV4XC53+g3pehIOnJZG/o/ON22Keg=; h=Date:From:To:cc:Subject:In-Reply-To:Message-ID:References:From; b=lS/khcgzGEkjobEBCFRBrjmjDdZI0pDf7DFetxT6OsDM/ti7y1Hck55Eopup/f49p 8cNETdJGjo+/RBg5f8WFmfl28MoWQ53Xu0wQgEUrZFVC7erMCibMC6JR2+DvUtSVD1 w7YPF9eNDyIRwEKP60vRyIbgOrBqMRNATvFQYiPoVhM0nn2cwfJfJhBNbapbDVCQ3s EA6jqMuVl8HYnvlOdfT0BJi+HCf2xdNKQ7AIk8p43yZ/XsI8atAoGA7OAZoNTqAYS7 SVOnfCU09ky4eKFV7OWL/tBm2cGyyi8zDZVQfVyNrWpHCwaWPADTAtKjYBSM+F3Co7 UzX1+NSoa2l9w== In-Reply-To: <87tu21hr7m.fsf@yahoo.com> Received-SPF: pass client-ip=95.142.160.155; envelope-from=gregory@heytings.org; helo=heytings.org X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.29 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-mx.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Xref: news.gmane.io gmane.emacs.devel:301209 Archived-At: >> Of course I did. That you did not read it is another thing. > > You did not. The only real technical argument you put forth was some > nonsense about FOR_EACH_TAIL being slow. > "Nonsense", again? Thank you! > > My change did not change any behavior, which is why I saw no purpose > discussing it at all. > It is telling that you "see no purpose" in discussing a change to code that was agreed upon after 300 posts in a bug report. > > All it did was rename and redocument a Lisp variable to remove > technicalities that are not of interest to our users. > That variable is of no interest whatsoever to our users. It is there only for debugging purposes, and is once again only meant to be used by the few users who understand subtle technicalities in the face realization code. >> And you revert without even reading or replying to the detailed >> explanation why that "improvement" was wrong. > > I replied. > Aha. That's your understanding of a "discussion", then: you say something, and act immediately, without waiting for a potential answer. And then claim that there was a "discussion" because you said something. Oh, in fact no, that's not even what happened: you reverted before saying anything. >> That's wrong. And you would have understood this if you had read the >> detailed explanation why your "improvement" is wrong. > > Really? What if a font entity is there, not a font spec? Or a font spec > from the Haiku font dialog? > You would not ask that question if you had read the explanation in https://debbugs.gnu.org/cgi/bugreport.cgi?bug=59347#331. And the fact you ask that question shows that you did not read it. > > 1. Nobody will ever remember to update the variable after the enum is > changed. > The relevant parts of that enum have not changed at all since they were introduced fifteen years ago. > > 2. Someone will fill the bitmask with random bogus that happens to do > nothing now, but will start to do something unexpected ``some time > later''. > Nobody should do that. The docstring clearly said: "There is no reason to change that value except for debugging purposes." > > 3. The maximum value of enum font_property_index will exceed what can > fit in 29 bits at some point in the future. > Oh yes, I see. FONT_SPEC_MAX is (and has always been) 15, but clearly, for a reason that hasn't happened in the past fifteen years, "at some point in the future", it will become 30, and that will be problematic on 32-bit computers. And you tell me that what I write is "nonsense". Anyway, I don't want to deal with this anymore. I probably spent about 50 hours on that bug, that's more than enough.