Re: [Orgmode] Re: [patch] LaTeX export of lists

2010-11-29 Thread Sébastien Mengin
Le dim. 28/11/10 (10:15:34 +0100), Nicolas Goaziou a écrit :
 
  This works perfectly for me. Thanks!
 
 It is available on master branch now.
 
 Thanks for pointing out this little annoyance.

Sorry I come too late but I would not like this kind of annoyance
avoided ;)

In Thomas' case, it is perfectly understandable, true.

But in general, I find it a lot better to leave a blank line between the
end of a list and the following paragraph in the LaTeX code -- easier to
read, I think.

Can your patch be set as an option or so, instead of the default
behaviour?

Thanks anyway,

-- 
Sébastien Mengin
Édition et logiciels libres
 Mise en page avec LaTeX 
http://edilibre.net
tél. : 06 84 88 49 17
jid. : sebastien-men...@jabber.org


___
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] Re: [patch] LaTeX export of lists

2010-11-29 Thread Nicolas Goaziou
Hello,

 Sébastien Mengin writes:

 Sorry I come too late but I would not like this kind of annoyance
 avoided ;)

 In Thomas' case, it is perfectly understandable, true.

 But in general, I find it a lot better to leave a blank line between
 the end of a list and the following paragraph in the LaTeX code --
 easier to read, I think.

 Can your patch be set as an option or so, instead of the default
 behaviour?

You just need to leave a blank line between your list and the next
paragraph. I don't see why an option should be needed here.

Or am I wrong?

Regards,

-- Nicolas

___
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


[Orgmode] Re: [patch] LaTeX export of lists

2010-11-28 Thread Nicolas Goaziou
Thomas,

Could you please test this patch and tell me if it breaks something?

Regards,

-- Nicolas

From 8fdd9ff682306cc5161f0b230d43493abff1d5e2 Mon Sep 17 00:00:00 2001
From: Nicolas Goaziou n.goaz...@gmail.com
Date: Sun, 28 Nov 2010 16:51:14 +0100
Subject: [PATCH] Exporting lists to LaTeX respects blank lines

* lisp/org-latex.el (org-export-latex-lists): do not add an
  unnecessary newline character after a list.
* lisp/org-list.el (org-list-bottom-point-with-indent): ensure bottom
  point is just after a non blank line.
---
 lisp/org-latex.el |2 +-
 lisp/org-list.el  |6 +-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/lisp/org-latex.el b/lisp/org-latex.el
index 91bf380..1a0aaf7 100644
--- a/lisp/org-latex.el
+++ b/lisp/org-latex.el
@@ -2383,7 +2383,7 @@ The conversion is made depending of STRING-BEFORE and 
STRING-AFTER.
   \n
   (match-string 1 res))
   t t res)))
-  (insert res \n
+  (insert res
 
 (defconst org-latex-entities
  '(\\!
diff --git a/lisp/org-list.el b/lisp/org-list.el
index 2290b4a..07a15e4 100644
--- a/lisp/org-list.el
+++ b/lisp/org-list.el
@@ -518,7 +518,11 @@ List ending is determined by the indentation of text. See
  (setq ind-ref ind)
  (forward-line 1))
 ((= ind ind-ref)
- (throw 'exit (point-at-bol)))
+ (throw 'exit (progn
+;; Ensure bottom is just after a
+;; non-blank line.
+(skip-chars-backward  \r\t\n)
+(min (point-max) (1+ (point-at-eol))
 ((looking-at #\\+begin_)
  (re-search-forward [ \t]*#\\+end_)
  (forward-line 1))
-- 
1.7.3.2

___
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


[Orgmode] Re: [patch] LaTeX export of lists

2010-11-28 Thread Thomas S. Dye

Aloha Nicolas,
On Nov 28, 2010, at 5:54 AM, Nicolas Goaziou wrote:


Thomas,

Could you please test this patch and tell me if it breaks something?

Regards,

-- Nicolas

0001-Exporting-lists-to-LaTeX-respects-blank-lines.patch


This works perfectly for me.  Thanks!

All the best,
Tom

___
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


[Orgmode] Re: [patch] LaTeX export of lists

2010-11-28 Thread Nicolas Goaziou

 This works perfectly for me. Thanks!

It is available on master branch now.

Thanks for pointing out this little annoyance.

Regards,

-- Nicolas

___
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