[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [PATCH] org-mac-link.el: Handle links to/from Adobe Acrobat.
From: |
Nicolas Goaziou |
Subject: |
Re: [O] [PATCH] org-mac-link.el: Handle links to/from Adobe Acrobat. |
Date: |
Thu, 10 Sep 2015 11:57:01 +0200 |
Hello,
Dmitri Makarov <address@hidden> writes:
> If you haven't pushed the patch yet, here is attached an updated
> patch, with a correction in the commit message, and a minor correction
> in one of the comments.
Thank you. Some comments follow.
> > On Thu, Sep 10, 2015 at 11:28 AM, Alan Schmitt <address@hidden> wrote:
>> LGTM (with a tiny caveat that I can fix: the first line of the commit
>> message should not end with a dot). Nicolas, can I push this?
Sure. However, it would be nice to fix the two minor issues below.
> From: Dmitri Makarov <address@hidden>
> Date: Wed, 9 Sep 2015 17:24:42 +0200
> Subject: [PATCH] org-mac-link.el: Handle links to/from Adobe Acrobat
>
> * contrib/lisp/org-mac-link.el: Add new link type "acrobat" to grab
> links to documents open in Adobe Acrobat application.
You need to mention new functions and variables in the commit message.
> +(defun as-get-acrobat-page-link ()
Wrong namespace. It should be prefixed with "org-mac-".
Regards,
--
Nicolas Goaziou