From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 25495 invoked by alias); 4 Oct 2015 00:49:25 -0000 Mailing-List: contact zsh-workers-help@zsh.org; run by ezmlm Precedence: bulk X-No-Archive: yes List-Id: Zsh Workers List List-Post: List-Help: X-Seq: 36767 Received: (qmail 14543 invoked from network); 4 Oct 2015 00:49:24 -0000 X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on f.primenet.com.au X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00 autolearn=ham autolearn_force=no version=3.4.0 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:message-id:date:in-reply-to:comments :references:to:subject:mime-version:content-type; bh=SAzD9w9C2Rtu6Bwdw3tWr/ZSEE1G26i2lrpA6ME10uU=; b=TV5QWSTQUy9egdtElHF0wIZ/1cBPVBBrLBM2pW8hOgxWmLQCSSzL9Wa4L32PRuz3HL zaAtpDI9ZKjuE4V/QKALn+w8dOlYkxJC3gsQOrwk7aWiUztVJA0qLodkO9+aOBYMuQ0y zS9VeQcyRsgOBzq44HKL6HOxV8aU/l3QFtYcpwRfr721bmC+kzXYBZkeCL9gLhw/74g9 UvgLE4oic7wxmMpQNhzKm4DYzs5om1TPDNNgvUKYdq2lj+sbu5ZuYabzdfrQB/F0UXbA Xc8Z3TOMys4xrbZEPQrT9eqBYz62oZDO9JHuwPMf/Y/zGWLwHggKPxzdpIdfRQyfuu7O 7Hlg== X-Gm-Message-State: ALoCoQm/BeRXZiVG6EqlmGFRGypMy6lXMFsmYTiwypuCAyzt5TLk4MykQBvfp3v2Qwt/98sNkVjT X-Received: by 10.182.16.165 with SMTP id h5mr13284785obd.48.1443919762279; Sat, 03 Oct 2015 17:49:22 -0700 (PDT) From: Bart Schaefer Message-Id: <151003174919.ZM31542@torch.brasslantern.com> Date: Sat, 3 Oct 2015 17:49:19 -0700 In-Reply-To: <20150930182948.GD2003@tarsus.local2> Comments: In reply to Daniel Shahaf "[PATCH] Add API wrapper to ${+_comps[...]}" (Sep 30, 6:29pm) References: <20150930182948.GD2003@tarsus.local2> X-Mailer: OpenZMail Classic (0.9.2 24April2005) To: zsh-workers@zsh.org Subject: Re: [PATCH] Add API wrapper to ${+_comps[...]} MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii On Sep 30, 6:29pm, Daniel Shahaf wrote: } } I wonder if this is useful enough to be added? (Why two separate patches both to compsys.yo?) I don't see any particular reason not to add it, but I also find no existence tests of $_comps[...] anywhere in the contributed functions or my own local startup files or functions, so it may indeed be only minimally useful. On the other hand there are tests for whether a particular function is defined. If that function is destined to be assigned to _comps[x] perhaps it would be better to check for _comps[x] already defined instead.