From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED!not-for-mail From: npostavs@users.sourceforge.net Newsgroups: gmane.emacs.bugs Subject: bug#23486: 25.0.93; Modules: features missing from make_function Date: Sun, 26 Mar 2017 23:57:20 -0400 Message-ID: <871stj4bj3.fsf@users.sourceforge.net> References: NNTP-Posting-Host: blaine.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: blaine.gmane.org 1490587046 4029 195.159.176.226 (27 Mar 2017 03:57:26 GMT) X-Complaints-To: usenet@blaine.gmane.org NNTP-Posting-Date: Mon, 27 Mar 2017 03:57:26 +0000 (UTC) User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) Cc: 23486@debbugs.gnu.org To: Philipp Stephani Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Mon Mar 27 05:57:22 2017 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by blaine.gmane.org with esmtp (Exim 4.84_2) (envelope-from ) id 1csLmS-0000VY-UZ for geb-bug-gnu-emacs@m.gmane.org; Mon, 27 Mar 2017 05:57:21 +0200 Original-Received: from localhost ([::1]:43987 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1csLmZ-0000xO-1F for geb-bug-gnu-emacs@m.gmane.org; Sun, 26 Mar 2017 23:57:27 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:44827) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1csLmE-0000pe-GF for bug-gnu-emacs@gnu.org; Sun, 26 Mar 2017 23:57:07 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1csLmA-00059P-Ml for bug-gnu-emacs@gnu.org; Sun, 26 Mar 2017 23:57:06 -0400 Original-Received: from debbugs.gnu.org ([208.118.235.43]:48440) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1csLmA-00059L-Iw for bug-gnu-emacs@gnu.org; Sun, 26 Mar 2017 23:57:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1csLmA-0001WL-Cv for bug-gnu-emacs@gnu.org; Sun, 26 Mar 2017 23:57:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: npostavs@users.sourceforge.net Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Mon, 27 Mar 2017 03:57:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 23486 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: Original-Received: via spool by 23486-submit@debbugs.gnu.org id=B23486.14905869675753 (code B ref 23486); Mon, 27 Mar 2017 03:57:02 +0000 Original-Received: (at 23486) by debbugs.gnu.org; 27 Mar 2017 03:56:07 +0000 Original-Received: from localhost ([127.0.0.1]:46634 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1csLlH-0001Uj-8N for submit@debbugs.gnu.org; Sun, 26 Mar 2017 23:56:07 -0400 Original-Received: from mail-it0-f47.google.com ([209.85.214.47]:38799) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1csLlF-0001UB-DM for 23486@debbugs.gnu.org; Sun, 26 Mar 2017 23:56:05 -0400 Original-Received: by mail-it0-f47.google.com with SMTP id y18so39545200itc.1 for <23486@debbugs.gnu.org>; Sun, 26 Mar 2017 20:56:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:from:to:cc:subject:references:date:in-reply-to:message-id :user-agent:mime-version; bh=9vmj0OGPv/cjeT8LhMEB2Q89usFpDddSkjlHKAAml9Q=; b=lLTg/2Ww+A2wr6OhLPYqiOAIz8NxO3oRJNvNH7kPvOPVdPQikvIumxP6wY/4BM6y8A 0dHPcQ/fcA0tfB0uQNw5NI6CG3peg+wWqpwdklEWj0lR6HM1NRfv4gLsUgHiFIj+JYtt DsmEp3ezy1gJ5kWwWUwXmHv9BVgOjDAaRM6U5ogrUlPCHOg5sVUKYSSRH35RTpvCtWIU vyIZkDpvPcVi+ccLA9XsdAXMxO1o9w4xvTcvF37PdPurmeUMTbaL7fKcXM9wDs1061DT 2ZMASh9T7Inb8+s4EUdaxIOZOjMcUr10V8/sxGLVW/WmuOHY03+wLJfwn+MDMxGrvo3n IYDg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:from:to:cc:subject:references:date :in-reply-to:message-id:user-agent:mime-version; bh=9vmj0OGPv/cjeT8LhMEB2Q89usFpDddSkjlHKAAml9Q=; b=tyElBvsDtc8DIqOHivx9kSsGpOa0D5N0zYGNaGcMYGRyHgL3LUDhuY+3+hJT5y8E3O VylPrZtFYqYYejIzKOYAQDZwv/Fdq3Jh+Kf4PJ7AaDq6KVtHQcMJUFPZ+XdtpSjKgp19 18qphwHeJHYx7qz3qUecdNboJX1hKNuIVOUnvNHnqLlV2oDnPOB+fOvJcP6zS1DeZdq/ UVn60cOxserHqBYsqaalY6q9T880LtlI5koIzJGQHm7Bcdxwrx/a75EQ78f5RZ20Xu+L sVwlwwNcDjebvVo4KcV5JvToK5NjK0BEgSAiAiuPKJoLrmGFQh+Kl8XdIdG9PDJdKSm2 8zbA== X-Gm-Message-State: AFeK/H3elaDSgcecEl2TbUJ/9SKU8ib5aIq+NLwQzdZtpmlqf++x6httYMTX90Toy0rNyQ== X-Received: by 10.36.89.211 with SMTP id p202mr8493540itb.97.1490586959812; Sun, 26 Mar 2017 20:55:59 -0700 (PDT) Original-Received: from zony ([45.2.7.65]) by smtp.googlemail.com with ESMTPSA id r197sm4872599itb.15.2017.03.26.20.55.59 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Sun, 26 Mar 2017 20:55:59 -0700 (PDT) In-Reply-To: (Philipp Stephani's message of "Mon, 09 May 2016 18:37:58 +0200") X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 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.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.org gmane.emacs.bugs:131001 Archived-At: Philipp Stephani writes: > As an alternative, if people feel this would require too many > parameters, I'd propose reverting the change that adds the documentation > string. A docstring without arglist is not very useful. We could also > remove the arity parameters and have the C function check the arity > itself. Looking at this a bit closer, I do think this adds too many parameters, and in particular, requiring to pass in names for positional parameters just makes no sense. The names are never used (except for displaying documentation). But removing the docstring is not great. IMO, the right solution here is to use a subr-like object instead of a lambda, as suggested in a FIXME in emacs-module.c: /* FIXME: Use a bytecompiled object, or even better a subr. */ Then the arity could be checked with `subr-arity' or similar.