Re: [Orgmode] [PATCH] Remove superfluous lambda.

2010-07-02 Thread Sebastian Rose
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.

2010-07-02 Thread John Wiegley
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.

2010-07-02 Thread Sebastian Rose
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.

2010-07-02 Thread John Wiegley
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.

2010-07-02 Thread Sebastian Rose
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.

2010-07-01 Thread Carsten Dominik

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