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.bugs Subject: bug#57499: Documentation bug in the docstring of set-face-attribute? Date: Wed, 31 Aug 2022 12:53:26 +0000 Message-ID: <534c9018d222586a161c@heytings.org> References: <534c9018d2adffda3e53@heytings.org> <831qswu0p4.fsf@gnu.org> <534c9018d2f372cd7462@heytings.org> <83tu5ssi35.fsf@gnu.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="18206"; mail-complaints-to="usenet@ciao.gmane.io" Cc: 57499@debbugs.gnu.org To: Eli Zaretskii Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Wed Aug 31 14:54:19 2022 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 1oTNEZ-0004Sj-7A for geb-bug-gnu-emacs@m.gmane-mx.org; Wed, 31 Aug 2022 14:54:19 +0200 Original-Received: from localhost ([::1]:51006 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1oTNEW-0006Lw-AY for geb-bug-gnu-emacs@m.gmane-mx.org; Wed, 31 Aug 2022 08:54:16 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:37996) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1oTNEI-0006KO-6Y for bug-gnu-emacs@gnu.org; Wed, 31 Aug 2022 08:54:02 -0400 Original-Received: from debbugs.gnu.org ([209.51.188.43]:48756) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1oTNEH-0001F1-Ro for bug-gnu-emacs@gnu.org; Wed, 31 Aug 2022 08:54:01 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1oTNEH-0001l6-N9 for bug-gnu-emacs@gnu.org; Wed, 31 Aug 2022 08:54:01 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Gregory Heytings Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Wed, 31 Aug 2022 12:54:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 57499 X-GNU-PR-Package: emacs Original-Received: via spool by 57499-submit@debbugs.gnu.org id=B57499.16619504106715 (code B ref 57499); Wed, 31 Aug 2022 12:54:01 +0000 Original-Received: (at 57499) by debbugs.gnu.org; 31 Aug 2022 12:53:30 +0000 Original-Received: from localhost ([127.0.0.1]:38505 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1oTNDl-0001kF-PN for submit@debbugs.gnu.org; Wed, 31 Aug 2022 08:53:29 -0400 Original-Received: from heytings.org ([95.142.160.155]:48798) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1oTNDj-0001k6-M0 for 57499@debbugs.gnu.org; Wed, 31 Aug 2022 08:53:28 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=heytings.org; s=20220101; t=1661950406; bh=vTTwaZybgrI9R9/9mhtYfXhcErYstM98/JTLTTAUJoQ=; h=Date:From:To:cc:Subject:In-Reply-To:Message-ID:References:From; b=nB9YBT/g1+I5W7u6alYwqz1Z2+oK3srAYuz7Oee7qlr/n8JOt42TPJjSLCJ3c6/P+ WqhEKVca2oKdA4Vkf6gbMcyZbVMJV3LJz/GoM647bch76rdhBBQQKYwq+vNTnkjoSe ft42cTgjX9cGjy0HiQRPoMATdcM0489VmwQmo78CuhK5AsyuJyTj2vSTOnUbmk1bof AnKiXarsxjyzLu5BiX8iWObdSzbgUDmPaTWUf1ffjdnX/RPxvsD+t9BmsDonjM2IGh o3GGOfEfqRzGOy12atlhEb0lpPxklha2FYxdiebWjCXd19bKfGgSs7pcrcjChf7BCc FYqrzQ4tslOKA== In-Reply-To: <83tu5ssi35.fsf@gnu.org> 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:241194 Archived-At: > > Or let me turn the table and ask: what do you want to change in the > current doc string? You want to tell that nil requires 2 calls, but > unspecified doesn't? > I see that this issue is repeatedly confusing users, so I would suggest something like: If FRAME is nil, set the attributes for all existing frames, as well as the default for new frames. If FRAME is t, change the default for new frames only. To reset the value of some attribute to `unspecified', you must use 'unspecified, not nil. This seems both much clearer than what we have now, and more correct, given that as far as I can tell (set-face-attribute 'some-face nil :some-attribute 'unspecified) changes the face attribute to "unspecified" on existing and future frame, and (set-face-attribute 'some-face t :some-attribute 'unspecified) changes the face attribute to "unspecified" on future frames only.