From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Drew Adams Newsgroups: gmane.emacs.bugs Subject: bug#51577: [External] : Re: bug#51577: 27.2; Regression: reproducible hang with face functions Date: Wed, 3 Nov 2021 18:58:23 +0000 Message-ID: References: <83tugtgo6z.fsf@gnu.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="19731"; mail-complaints-to="usenet@ciao.gmane.io" Cc: "51577@debbugs.gnu.org" <51577@debbugs.gnu.org> To: Eli Zaretskii Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Wed Nov 03 19:59:12 2021 Return-path: Envelope-to: geb-bug-gnu-emacs@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 1miLTb-0004sp-MS for geb-bug-gnu-emacs@m.gmane-mx.org; Wed, 03 Nov 2021 19:59:11 +0100 Original-Received: from localhost ([::1]:33000 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1miLTa-0006aO-2Y for geb-bug-gnu-emacs@m.gmane-mx.org; Wed, 03 Nov 2021 14:59:10 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:39490) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1miLTS-0006a6-50 for bug-gnu-emacs@gnu.org; Wed, 03 Nov 2021 14:59:02 -0400 Original-Received: from debbugs.gnu.org ([209.51.188.43]:57954) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1miLTR-0004T0-Sn for bug-gnu-emacs@gnu.org; Wed, 03 Nov 2021 14:59:01 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1miLTR-0004V0-Rp for bug-gnu-emacs@gnu.org; Wed, 03 Nov 2021 14:59:01 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Drew Adams Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Wed, 03 Nov 2021 18:59:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 51577 X-GNU-PR-Package: emacs Original-Received: via spool by 51577-submit@debbugs.gnu.org id=B51577.163596590917228 (code B ref 51577); Wed, 03 Nov 2021 18:59:01 +0000 Original-Received: (at 51577) by debbugs.gnu.org; 3 Nov 2021 18:58:29 +0000 Original-Received: from localhost ([127.0.0.1]:41261 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1miLSv-0004Tn-D5 for submit@debbugs.gnu.org; Wed, 03 Nov 2021 14:58:29 -0400 Original-Received: from mx0b-00069f02.pphosted.com ([205.220.177.32]:46344) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1miLSt-0004Tf-Uv for 51577@debbugs.gnu.org; Wed, 03 Nov 2021 14:58:28 -0400 Original-Received: from pps.filterd (m0246632.ppops.net [127.0.0.1]) by mx0b-00069f02.pphosted.com (8.16.1.2/8.16.1.2) with SMTP id 1A3Hsej9020540; Wed, 3 Nov 2021 18:58:27 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=from : to : cc : subject : date : message-id : references : in-reply-to : content-type : content-transfer-encoding : mime-version; s=corp-2021-07-09; bh=RkjSHI3Qa1B/BNkSUHDZQDEstoACHlYuv/tEu34VT/M=; b=fyZY0TF1qd0Ru93tADh6GbaELu1tD8dcrkHtQ4NH5v0kfxIbmLPhPkGjYPxX0eW0QFIl fiS7oEHuS4XYlfP3wWUjQE/flsZx/W/Ryw38KcmKeV1lWAUF+Et8ZASe8hE62yGrpEYq CfimikDud1IXmn8vJHiqWa58ehWqTcrUePN2FvXE2xDV2ShjMkL5ynpiOARAv1bzoGY5 LcJBoxzOeajYR6xFU6LK2X/NZYcJbsyf8qGZWi83JjRAwGAvHPmS7Z7NfcITukGnVKrm wlPatUkhoMyVrn0G4LbYPNV5bU98ASGZHhm+TfrPOVGk8r+LbTHJE4Psd8vFnbZvZS/A qQ== Original-Received: from userp3020.oracle.com (userp3020.oracle.com [156.151.31.79]) by mx0b-00069f02.pphosted.com with ESMTP id 3c3n9xm403-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Wed, 03 Nov 2021 18:58:26 +0000 Original-Received: from pps.filterd (userp3020.oracle.com [127.0.0.1]) by userp3020.oracle.com (8.16.1.2/8.16.1.2) with SMTP id 1A3IuL3C038641; Wed, 3 Nov 2021 18:58:25 GMT Original-Received: from nam10-mw2-obe.outbound.protection.outlook.com (mail-mw2nam10lp2108.outbound.protection.outlook.com [104.47.55.108]) by userp3020.oracle.com with ESMTP id 3c1khw2axt-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Wed, 03 Nov 2021 18:58:25 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=EniAlVphBwqa4RbxkI2Bf+IIAoeZNyYE0mqBoqiz4czbK5gVUZvTUOwVOsSAIRazwt3ZJUG5GjYclQr+O6GKVb+QX+JxCwaOiYDUzpXAWQYbbpThCcbb2L7WbiMXg5IEBdf3T5siGQ1egjHyJkE5QW0YlT9R7j/RefMtOZPqLQBIhDAIidbMw/a+gu8pSJZx6QiDoUmJBDmUTnLxyy8rW3cvPO+3Wsb1aNbgBV2kf3OriOtFW1er7Rd5PJ6SAcxuFnNkloYf5VnKb3egzZEsspWyfl3EBKJPYFcAaEkHuoM30LASTvLZafyLuda71cuNTBmGCd9oR6BrwNPrHaSLkw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=RkjSHI3Qa1B/BNkSUHDZQDEstoACHlYuv/tEu34VT/M=; b=lt+j9Bfwq1f6Y+MzynMxF0BXLfwkNxiPyVeRiNCQEATW1nOAd6NgVcHVPHUnny3H4ugAY6z6RaxNKoo/0Yps396y05xKN35wFdWhMasOkKz+t7RXdL8K+942TfLzOLDOMiogWVV3X2JUT867MjX/tmSBmVQl/H/W9P/n3xtMhtYlPwFv5hXFetEkytC7RfFfEUKCqHUmmKlCEop/kwgi+he5loZbwxxsP2lggRj6uORSkq8X9I2cj7289EpJOxp5he9+YRYIhfpf3O+StTppbgoWURpxJfGnsyypcCfUhQRfmsI/187uY0dSY0oKCcE5+XFaRL3v3j1lILbxVyB/XA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oracle.com; dmarc=pass action=none header.from=oracle.com; dkim=pass header.d=oracle.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.onmicrosoft.com; s=selector2-oracle-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=RkjSHI3Qa1B/BNkSUHDZQDEstoACHlYuv/tEu34VT/M=; b=blhoWzPXj9beDY+RZGwrRLPcZrKgSk4uoyEruOQ/MAdYr5wgovYg7H08Cq+Y6q/CKhOQ/82ruTAwifoOoIoXT7RxStF/l3Fv2R6PUt0qMGHZFyyNNO77Vb+0CSPeGdHfE75uoOIO1ciTCBWqHOtloqzQ+o2oVOMV+UPpN6p08Jk= Original-Received: from SJ0PR10MB5488.namprd10.prod.outlook.com (2603:10b6:a03:37e::19) by BYAPR10MB2967.namprd10.prod.outlook.com (2603:10b6:a03:87::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4649.18; Wed, 3 Nov 2021 18:58:23 +0000 Original-Received: from SJ0PR10MB5488.namprd10.prod.outlook.com ([fe80::b0d3:555d:98c3:62e3]) by SJ0PR10MB5488.namprd10.prod.outlook.com ([fe80::b0d3:555d:98c3:62e3%4]) with mapi id 15.20.4649.019; Wed, 3 Nov 2021 18:58:23 +0000 Thread-Topic: [External] : Re: bug#51577: 27.2; Regression: reproducible hang with face functions Thread-Index: AdfQWxo0P/xGbEZqSDmr6NZcWoIO6QAeqHpMAAM7RcA= In-Reply-To: <83tugtgo6z.fsf@gnu.org> Accept-Language: en-US Content-Language: en-US authentication-results: gnu.org; dkim=none (message not signed) header.d=none;gnu.org; dmarc=none action=none header.from=oracle.com; x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 0a372897-fcf1-410e-1ad7-08d99efbe91b x-ms-traffictypediagnostic: BYAPR10MB2967: x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:9508; x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: QkTXar5gUaQqrATIOfeJHE8Ujerb4dGYhrLid+Ld0+7CtwHAcS+uVPcBmqSESIGpBgLRHVK3dWW6+Pdw6I/itcYEV04lRMX/F+Kp4ynXvQ+lvVQsvcXcslBLgiIhW/7B4YLZJzMDHj2NAK59WP1To19EsVgHdFXGeudRLEh7ZaZp1ADC51Rfzj60cuRxEZfCyFZw1+IuRJCsLxb2g4xgv0zpkY55MWeHoEWSRA4IflRcyKcMSW4WcdEvpAXovf5orxXOVNV9hgkCpNgOYiYpHoA3RPDb8L404mQ+fvR4bI5UJAqve74sqOI5/Q0pXoVq6mKLAORSmKVMYlGOp99QlfUJRa64y/Yxu0a8gr1dEzGXXPEp0hjevjo3DJXpV+S08LBzATuSuOauR0OFwuVp1VDG5OLOgGlfagwOm87haoiNqd+YTKfjXnkUvhv5emTcXIhq5/YjXAl3Jkk5OuTvn/J/BMY10LQOcDdlAswx1eqpI7OCIPs0fr4QuVlKFrprIMfHnXsQ9Ad90in37OR6/WK9goJFxMuBY943UmOiu0GkwVtnYIShoBaTN/aMZfCcJ22ykT9OI2EnjEXlbdc25sN7RcjdTMTpkZNnoapaSgWQzIESFf1tk7b14ll/lXksYvnj7Pg4ppPeLhG30NUua6JETQN1wt7AxL5X/WP6OtOIoM9TtMKqzvx9MB6wSohR84hf9bHy3FlrG1jhELjgtA== x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:SJ0PR10MB5488.namprd10.prod.outlook.com; PTR:; CAT:NONE; SFS:(366004)(8936002)(86362001)(186003)(52536014)(2906002)(316002)(71200400001)(26005)(83380400001)(55016002)(5660300002)(64756008)(66476007)(66946007)(508600001)(33656002)(4326008)(9686003)(6506007)(6916009)(38070700005)(8676002)(66446008)(44832011)(7696005)(66556008)(38100700002)(76116006)(122000001); DIR:OUT; SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: 76LiEkeeJ+IwRjPE5bhh9Z6lQvpdRacp0aknXdKYA7um2TbAQfygsokqGN+r8z1L25tczA3axLBWEnuR40T5VhNXn8OVzuPbTIeUtDNtSJL8JliwEtqdxghUcL9Bn1fX8YtQ8TPFUKAAxnNbkIxproYqNE60YsYyCl1hLDa9hvhJlmeVrUDgHdO5b38dSsiZhXW8SaYkVyAYAWVAJqR8qbPM8CIDFs6UvfEqKb6noOQ9PBGAaqJB7LjcSd7k6xjK/aNcgUShLRgDb06rhME6X6ruA5JFKj5KFDVD0XQoxs9DYJXdAtISRA1VYkxkEQDQN+RjH6zhPcWd9oQ+WImH5kGpTlkNGhs11B2WtEVp3HzbsCfhlgCdA6tb6Wd2YVY3GwluTe0vHwX1xsEr9z4ZXKHa1spY+T4v41oxKRcAIIHB2VZyD9hCG2pckdOLcSjbZV7OBL2Fhuh0iOXlgmN0ZK4T+AZluqG4DP58PSB02jRt56uSKkxMZVPQQD3FapTkKvKat22tN9JnwHlpEpXWbnwX1Zltov8Ud/9F6MeEbVA5FovlnW2MuFElsP4sECgKIrojdC4StKGCx9f7B6TneOa5OVR0Qb1aUXXSQnYC6L9GcQIx5iqPIDN6n6XDjOaCUJWSum2f7YPf9mGdUp/H1xbxlAkF7Kai5vZgMYOLdvzqQpKdDwliNqMrE8scPNKlvKc96PzX5S0ttyHP2iP703H0c2ctZ3IzqtmDJbyRtWa3Jo5gVHOdipkoqN hMVKnptiv0B+Yvj2n5PVDXkxckPVfTbbB80M15ySSJ5Es8z2djtv4o8J/Uu59UeD4gB943+QpFSlWeZKIhf+Tt2yt8WrdUKjLR X-OriginatorOrg: oracle.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: SJ0PR10MB5488.namprd10.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 0a372897-fcf1-410e-1ad7-08d99efbe91b X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Nov 2021 18:58:23.2920 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: q1NIH5ZTFo7VAisMdp5TeVZUsV7pCbYC1spaWx652jtm/HwYB+XvGje2U33lCSQ4TBxpuZ7pei3YoFxZgPPYJQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR10MB2967 X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10157 signatures=668683 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 malwarescore=0 phishscore=0 bulkscore=0 spamscore=0 adultscore=0 suspectscore=0 mlxlogscore=665 mlxscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2110150000 definitions=main-2111030100 X-Proofpoint-ORIG-GUID: wqJnFintE4V4r2Du5sCi0daJv3mJ3crx X-Proofpoint-GUID: wqJnFintE4V4r2Du5sCi0daJv3mJ3crx X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.io gmane.emacs.bugs:218894 Archived-At: > > (defface alt-region '((t :background "gray70" :inherit region)) > > "..." :group 'faces) > > > > ;; `selected-frame' here could be anything, so this is dicey. > > ;; But that's not important for the bug. > > (defvar orig-region-atts (face-all-attributes 'region (selected- > frame))) > > > > (defun foo (&optional arg) > > (interactive "P") > > (let* ((frame (selected-frame)) > > (alist (if arg > > (face-all-attributes 'alt-region frame) > > orig-region-atts)) > > (alist (cons (cons :font 'unspecified) alist)) > > (plist ())) > > (message "BEFORE LOOP, ALIST: %S" alist) > > (while alist > > (push (caar alist) plist) > > (push (cdar alist) plist) > > (setq alist (cdr alist))) > > (setq plist (nreverse plist)) > > (message "> LOOP. PLIST: %S" plist) > > (apply #'set-face-attribute 'region frame plist))) >=20 > You set the region face to inherit from itself, and you expect that to > work without causing an infloop when Emacs tries to resolve some face > attribute? Yes. I expected Emacs to act as it has in the past, and ignore such an :inherit as a no-op. > If FACE1 has some attribute 'unspecified', but inherits > from FACE2, Emacs will try to go up the inheritance chain to see if > some of the parent faces specifies that attribute. If FACE1 inherits > from itself, going up the inheritance chain will never end. Only if the implementation doesn't recognize the inheritance loop. > IOW, it's a cockpit error. You can look at it that way. But I'd expect that inheritance of a face from itself would be a no-op. That makes :inherit more flexible/usable. And that's exactly what the case was in Emacs 26. The `region' face spec with `:inherit region' is innocuous in Emacs 26. Why should the code now chase its tail down an infinite rabbit hole? Is this really by design? Was something gained by this code change? Or is this just accidental collateral damage? Not rhetorical questions. It seems like a design change has been made, but with no announcement. I see nothing in NEWS, when searching for "face" or "inherit".