From mboxrd@z Thu Jan  1 00:00:00 1970
From: Eric Schulte <schulte.eric@gmail.com>
Subject: Re: [patch] bring back the "multiline" option for latex tables
Date: Mon, 13 Jun 2011 14:10:29 -0700
Message-ID: <87sjrdjv3u.fsf@gmail.com>
References: <87boyhttco.fsf@gmail.com>
Mime-Version: 1.0
Content-Type: text/plain
Return-path: <emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org>
Received: from eggs.gnu.org ([140.186.70.92]:33419)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <schulte.eric@gmail.com>) id 1QWEPC-0000yQ-TJ
	for emacs-orgmode@gnu.org; Mon, 13 Jun 2011 17:10:44 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <schulte.eric@gmail.com>) id 1QWEPB-0006I8-55
	for emacs-orgmode@gnu.org; Mon, 13 Jun 2011 17:10:42 -0400
Received: from mail-pz0-f41.google.com ([209.85.210.41]:64784)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <schulte.eric@gmail.com>) id 1QWEPA-0006Hx-Ku
	for emacs-orgmode@gnu.org; Mon, 13 Jun 2011 17:10:40 -0400
Received: by pzk4 with SMTP id 4so2770514pzk.0
	for <emacs-orgmode@gnu.org>; Mon, 13 Jun 2011 14:10:39 -0700 (PDT)
In-Reply-To: <87boyhttco.fsf@gmail.com> (Eric Schulte's message of "Wed, 01
	Jun 2011 10:21:59 -0600")
List-Id: "General discussions about Org-mode." <emacs-orgmode.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/emacs-orgmode>,
	<mailto:emacs-orgmode-request@gnu.org?subject=unsubscribe>
List-Archive: </archive/html/emacs-orgmode>
List-Post: <mailto:emacs-orgmode@gnu.org>
List-Help: <mailto:emacs-orgmode-request@gnu.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/emacs-orgmode>,
	<mailto:emacs-orgmode-request@gnu.org?subject=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: Eric Schulte <schulte.eric@gmail.com>
Cc: Org Mode <emacs-orgmode@gnu.org>

If there are no objections I'm going to apply this patch.  It is
relatively small so hopefully any new bugs should be easily
identifiable.

Best -- Eric

Eric Schulte <schulte.eric@gmail.com> writes:

> Hi,
>
> The following patch make the behavior of LaTeX table exporter more
> closely resemble the documented behavior (at least to my mind).  I'm
> hesitant to apply this patch myself as it applies to org-latex.el which
> is outside of my area of expertise.
>
> Please apply this if it looks reasonable.
>
> Thanks -- Eric
>
>
> From fc8700f52c2fbfb1ec601e3bbb32c37e38a4cfbb Mon Sep 17 00:00:00 2001
> From: Eric Schulte <schulte.eric@gmail.com>
> Date: Wed, 1 Jun 2011 10:18:34 -0600
> Subject: [PATCH] replacing the "multicolumn" option for latex tables
>
>   This brings the behavior inline with the documentation.
>
> * lisp/org-latex.el (org-export-latex-tables): Check for the
>   "multiline" option and set the `floatp' option to true when
>   multicolumn tables are requested.
> ---
>  lisp/org-latex.el |    5 +++--
>  1 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/lisp/org-latex.el b/lisp/org-latex.el
> index e2c74ae..0d64f68 100644
> --- a/lisp/org-latex.el
> +++ b/lisp/org-latex.el
> @@ -1838,7 +1838,8 @@ The conversion is made depending of STRING-BEFORE and STRING-AFTER."
>                    longtblp (and attr (stringp attr)
>                                  (string-match "\\<longtable\\>" attr))
>  		  tblenv (if (and attr (stringp attr)
> -				  (string-match (regexp-quote "table*") attr))
> +				  (or (string-match (regexp-quote "table*") attr)
> +				      (string-match "\\<multicolumn\\>" attr)))
>  			     "table*" "table")
>  		  tabular-env
>  		  (if (and attr (stringp attr)
> @@ -1851,7 +1852,7 @@ The conversion is made depending of STRING-BEFORE and STRING-AFTER."
>                    align (and attr (stringp attr)
>                               (string-match "\\<align=\\([^ \t\n\r]+\\)" attr)
>                               (match-string 1 attr))
> -                  floatp (or caption label)
> +                  floatp (or caption label (string= "table*" tblenv))
>  		  placement     (if (and attr
>  					 (stringp attr)
>  					 (string-match "[ \t]*\\<placement=\\(\\S-+\\)" attr))
> -- 
> 1.7.4.1

-- 
Eric Schulte
http://cs.unm.edu/~eschulte/