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 09:09:43 +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> 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="19424"; 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 10:12:38 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 1p4erT-0004nz-MS for ged-emacs-devel@m.gmane-mx.org; Mon, 12 Dec 2022 10:12:35 +0100 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1p4epb-0002Da-Og; Mon, 12 Dec 2022 04:10:39 -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 1p4eox-00022V-TH for emacs-devel@gnu.org; Mon, 12 Dec 2022 04:10:11 -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 1p4eos-0000IG-S9 for emacs-devel@gnu.org; Mon, 12 Dec 2022 04:09:56 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=heytings.org; s=20220101; t=1670836183; bh=veDQyVY61eHHgm/Gd8NQCgIoiBIejK2W86VPXTX+t/E=; h=Date:From:To:cc:Subject:In-Reply-To:Message-ID:References:From; b=nYkHrp7BsuMnfddr+pmkkESHoEl3cbo6V1w31Bg1nVt5TryxUGP4RwYU/YH2LNl8y omHQMNji6ezNZFM2S0i43VB1vmCQ/XtnyA0qUlIcR4J20psvvmMN/nyqpYWwrufQQm WYriBDoXN/w713XDlW+qfCI33fW6qcGCu8Zoeb26rKz5h6dtQT0zEW4Qgr6PtVhC/u QBTOnu24p7KTYVK9caYxLOh4vC3gg3QdsEWKSmISUv1hzLJNwrjUm2ZeBW59O4/ilE rpDyYzmPSEkN1xiIhTwaZ1vWcjYhsgxU+vqgPuaXetwrbrf/4iExq3HVHGPL/DSXiM XP+v1kyIAHeTA== In-Reply-To: <87359ljq04.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:301199 Archived-At: >> Don't do that! You did not answer ANY part of what I said. Of course I did. That you did not read it is another thing. > > Btw, for posterity's sake, I undid this revert for two reasons: > What kind of development practice is this? You "improve" code in a complex area of the Emacs code base that was agreed upon after a long discussion only a couple of hours after it was pushed, without asking anyone whether what you want to do is okay? And you revert without even reading or replying to the detailed explanation why that "improvement" was wrong. > > - unsetting the "extra" attribute is not safe on the Haiku port. > That's wrong. And you would have understood this if you had read the detailed explanation why your "improvement" is wrong. > > - the bitmask variable is a real nusiance for anyone trying to debug > Emacs or change the layout of the font attribute index enumerator. > It isn't, and it is not supposed to be modified on a daily basis. > > Just because a bug has been closed does NOT mean the change in it is no > longer subject to scrutiny. I don't follow bug reports that aren't > related to X, which means I (and possibly many others) only see changes > as they arrive on emacs-diffs. Which means that by the time the bug is > closed, no, the discussion is not ``over'', and other people still have > a chance to make changes to problems they see as they see them (a > bitmask depending on the internal layout of an enum exposed to Lisp is > definitely one such problem, so is unsetting `:extra'.) > Nobody said the discussion was over (although I was hoping it was). But what you did is not a "discussion", it is the exact opposite of a "discussion": these are misguided changes introduced in the release branch _without any discussion_.