Re: [Orgmode] [PATCH] Remove superfluous lambda.
David Maus dm...@ictsoc.de writes: Carsten Dominik wrote: Hmm, the catcher did not see this. Why? He did: http://patchwork.newartisans.com/patch/105/ -- patch is already applied. Where can I see that? I read Accepted which is not Applied, is it? And there are accepted patches, that are not applied. E.g. http://patchwork.newartisans.com/patch/73/ The diff against the current head (8da31057eb0952889858c): diff --git a/lisp/org-capture.el b/lisp/org-capture.el index 8c887ce..f38a78c 100644 --- a/lisp/org-capture.el +++ b/lisp/org-capture.el @@ -924,6 +924,8 @@ Point will remain at the first line after the inserted text. (org-capture-put :key (car entry) :description (nth 1 entry) :target (nth 3 entry)) (let ((txt (nth 4 entry)) (type (or (nth 2 entry) 'entry))) +(when (file-exists-p txt) + (setq txt (org-file-contents txt))) (when (or (not txt) (not (string-match \\S- txt))) ;; The template may be empty or omitted for special types. ;; Here we insert the default templates for such cases. Is it possible to link to the commit a patch was applied? Sebastian ___ Emacs-orgmode mailing list Please use `Reply All' to send replies to the list. Emacs-orgmode@gnu.org http://lists.gnu.org/mailman/listinfo/emacs-orgmode
Re: [Orgmode] [PATCH] Remove superfluous lambda.
On Jul 2, 2010, at 9:21 AM, Sebastian Rose wrote: I read Accepted which is not Applied, is it? Accepted = Applied. Is it possible to link to the commit a patch was applied? After today, patches which are merged in using pw will note the resulting commit. John ___ Emacs-orgmode mailing list Please use `Reply All' to send replies to the list. Emacs-orgmode@gnu.org http://lists.gnu.org/mailman/listinfo/emacs-orgmode
Re: [Orgmode] [PATCH] Remove superfluous lambda.
John Wiegley jwieg...@gmail.com writes: On Jul 2, 2010, at 9:21 AM, Sebastian Rose wrote: I read Accepted which is not Applied, is it? Accepted = Applied. http://patchwork.newartisans.com/patch/73/ was never aplied. I just diffed my branch against the current head. Sebastian ___ Emacs-orgmode mailing list Please use `Reply All' to send replies to the list. Emacs-orgmode@gnu.org http://lists.gnu.org/mailman/listinfo/emacs-orgmode
Re: [Orgmode] [PATCH] Remove superfluous lambda.
On Jul 2, 2010, at 9:49 AM, Sebastian Rose wrote: John Wiegley jwieg...@gmail.com writes: On Jul 2, 2010, at 9:21 AM, Sebastian Rose wrote: I read Accepted which is not Applied, is it? Accepted = Applied. http://patchwork.newartisans.com/patch/73/ was never aplied. I just diffed my branch against the current head. Anyone know who accepted this? They didn't make themselves the delegate when they accepted it. I'll mark it as New again. John ___ Emacs-orgmode mailing list Please use `Reply All' to send replies to the list. Emacs-orgmode@gnu.org http://lists.gnu.org/mailman/listinfo/emacs-orgmode
Re: [Orgmode] [PATCH] Remove superfluous lambda.
John Wiegley jwieg...@gmail.com writes: On Jul 2, 2010, at 9:49 AM, Sebastian Rose wrote: John Wiegley jwieg...@gmail.com writes: On Jul 2, 2010, at 9:21 AM, Sebastian Rose wrote: I read Accepted which is not Applied, is it? Accepted = Applied. http://patchwork.newartisans.com/patch/73/ was never aplied. I just diffed my branch against the current head. Anyone know who accepted this? They didn't make themselves the delegate when they accepted it. I'll mark it as New again. Thanks a bunch! Sebastian ___ Emacs-orgmode mailing list Please use `Reply All' to send replies to the list. Emacs-orgmode@gnu.org http://lists.gnu.org/mailman/listinfo/emacs-orgmode
Re: [Orgmode] [PATCH] Remove superfluous lambda.
Hmm, the catcher did not see this. Why? - Carsten On Jul 1, 2010, at 8:02 PM, David Maus wrote: * org-feed.el (org-feed-unescape): Remove superfluous lambda. --- lisp/org-feed.el |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/lisp/org-feed.el b/lisp/org-feed.el index 3edcf1a..cda7368 100644 --- a/lisp/org-feed.el +++ b/lisp/org-feed.el @@ -271,8 +271,7 @@ have been saved. (defun org-feed-unescape (s) Unescape protected entities in S. (let ((re (concat \\( - (mapconcat (lambda (e) -(car e)) xml-entity-alist \\|) + (mapconcat 'car xml-entity-alist \\|) \\);))) (while (string-match re s) (setq s (replace-match -- 1.7.1 ___ Emacs-orgmode mailing list Please use `Reply All' to send replies to the list. Emacs-orgmode@gnu.org http://lists.gnu.org/mailman/listinfo/emacs-orgmode - Carsten ___ Emacs-orgmode mailing list Please use `Reply All' to send replies to the list. Emacs-orgmode@gnu.org http://lists.gnu.org/mailman/listinfo/emacs-orgmode