From: Samuel Loury <konubinix@gmail.com>
To: OrgMode ML <emacs-orgmode@gnu.org>
Subject: [PATCH] Make the regexp matching the beginning of a table more explicit
Date: Wed, 20 Aug 2014 14:48:49 +0200 [thread overview]
Message-ID: <87oavfqqby.fsf@konixwork.incubateur.ens-lyon.fr> (raw)
[-- Attachment #1.1: Type: text/plain, Size: 677 bytes --]
Hi,
I would like to provide a very tiny change in the regexp matching a
beginning of table in org-table.el.
Its current value is "^\\(\\*+ \\)\\|[ \t]*|". The second part of the
regexp ("[ \t]*|") indicates that it matches any | character, following
spaces or not. This means that the "[ \t]" is useless.
Actually, it matches my :tags expression in the #+BEGIN: clocktable part
because its value is :tags "@home|@computer".
I think that Carsten Dominik wanted the | character to be matched only
when at the beginning of a line (modulo indentation spaces).
I then added the ^ character in this part of the regexp to make it work
that way.
Please find the patch attached.
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1.2: 0001-Make-the-regexp-matching-the-beginning-of-a-table-mo.patch --]
[-- Type: text/x-diff, Size: 1878 bytes --]
From 04fe96d8782719b2ef90afd16c6b3dcfbd9349c6 Mon Sep 17 00:00:00 2001
From: Konubinix <konubinix@gmail.com>
Date: Wed, 20 Aug 2014 14:32:47 +0200
Subject: [PATCH] Make the regexp matching the beginning of a table more
explicit
* org-table.el (org-table-get-remote-range): Make the regexp matching the
beginning of a table more explicit, all | characters were matched, even those
contained into the :tags value (like in :tags "@home|@computer"). This caused
the | character to be interpreted as the beginning of a line. Now, the | is
matched only if it is at the beginning at a line (modulo beginning spaces).
---
lisp/org-table.el | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/lisp/org-table.el b/lisp/org-table.el
index d1609f9..5654783 100644
--- a/lisp/org-table.el
+++ b/lisp/org-table.el
@@ -42,7 +42,7 @@
(string backend &optional body-only ext-plist))
(declare-function aa2u "ext:ascii-art-to-unicode" ())
(declare-function calc-eval "calc" (str &optional separator &rest args))
-
+
(defvar orgtbl-mode) ; defined below
(defvar orgtbl-mode-menu) ; defined when orgtbl mode get initialized
(defvar constants-unit-system)
@@ -2768,7 +2768,7 @@ not overwrite the stored one."
(user-error "Invalid field specifier \"%s\""
(match-string 0 form)))
(setq form (replace-match formrpl t t form)))
-
+
(if lispp
(setq ev (condition-case nil
(eval (eval (read form)))
@@ -5053,7 +5053,7 @@ list of the fields in the rectangle."
(widen)
(goto-char loc)
(forward-char 1)
- (unless (and (re-search-forward "^\\(\\*+ \\)\\|[ \t]*|" nil t)
+ (unless (and (re-search-forward "^\\(\\*+ \\)\\|^[ \t]*|" nil t)
(not (match-beginning 1)))
(user-error "Cannot find a table at NAME or ID %s" name-or-id))
(setq tbeg (point-at-bol))
--
2.0.1
[-- Attachment #1.3: Type: text/plain, Size: 99 bytes --]
--
Konubinix
GPG Key : 7439106A
Fingerprint: 5993 BE7A DA65 E2D9 06CE 5C36 75D2 3CED 7439 106A
[-- Attachment #2: Type: application/pgp-signature, Size: 472 bytes --]
next reply other threads:[~2014-08-20 12:49 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-20 12:48 Samuel Loury [this message]
2014-08-22 13:44 ` [PATCH] Make the regexp matching the beginning of a table more explicit Bastien
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87oavfqqby.fsf@konixwork.incubateur.ens-lyon.fr \
--to=konubinix@gmail.com \
--cc=emacs-orgmode@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.