From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nicolas Goaziou Subject: Re: org-player and switch to lexical binding in org.el Date: Fri, 29 Jan 2016 14:32:17 +0100 Message-ID: <87oac4a4v2.fsf@nicolasgoaziou.fr> References: <877fj8j5wx.fsf@nicolasgoaziou.fr> Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:37415) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aP980-0000ud-TT for emacs-orgmode@gnu.org; Fri, 29 Jan 2016 08:30:21 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aP97x-0004jg-LH for emacs-orgmode@gnu.org; Fri, 29 Jan 2016 08:30:20 -0500 Received: from relay4-d.mail.gandi.net ([217.70.183.196]:60010) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aP97x-0004jc-E1 for emacs-orgmode@gnu.org; Fri, 29 Jan 2016 08:30:17 -0500 In-Reply-To: (Michael Brand's message of "Thu, 28 Jan 2016 12:37:00 +0100") List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org Sender: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org To: Michael Brand Cc: Org Mode , Paul Sexton Hello, Michael Brand writes: > On Sun, Jan 17, 2016 at 9:40 PM, Nicolas Goaziou wrote: > >> In the current case, you need to use match string: >> >> (add-to-list 'org-file-apps >> (cons (concat org-player-file-extensions-regexp >> "::\\([0-9]+:[0-9]+\\(:[0-9]+\\)?\\)") >> (lambda (file link) >> (org-player-play-file file (match-string 1 link))))) >> >> WDYT? > > I am still not able to get your suggestion that Paul has implemented > in the meantime in org-player.el 1.0.1 to work. Now I am trying to > understand what I am missing with the lambda. > > Has the (eval cmd) in org-open-file to be extended somehow to the > variant (funcall cmd) as lambda evaluates to itself? My suggestion was hypothetical, and not yet implemented. No wonder it doesn't work. If you think this change sounds reasonable, I can implement it, tho. However, there is some backward incompatibility involved. The current solution, i.e., using `eval' only provides `file' symbol. Regards, -- Nicolas Goaziou