ndwarshuis
/
org-mode
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity
org-mode/lisp
History
Eric Schulte 08a7f6a6df fix infinite loop in csv table import 
* lisp/org-table.el (org-table-convert-region): don't continue csv
importation which the point catches the end, this fixes an infinite
loop which was caused by the (point) never catching up with the
"end" marker

Hi,

I've just noticed that I get an infinite loop when importing csv tables
into Org-mode tables.  For example calling

  (org-table-convert-region (point-min) (point-max) '(4))

from inside of a simple text file containing the following

--8<---------------cut here---------------start------------->8---
1,2,3
1
2
3
5
--8<---------------cut here---------------end--------------->8---

results in an infinite loop.  The attached patch fixes this issue [1],
however it seems weird that this would just surface now, given that the
code in question has been in the repo since last fall

,----
| commit 59c9c4cdd4
| Author: Carsten Dominik <carsten.dominik@gmail.com>
| Date:   Mon Oct 26 12:31:16 2009 +0100
|
|     Correctly interpret CVS tables with quoted fields
|
|     The csv parser was very primitive, ignoring quoted fields.  This is
|     now fixed.
`----

Has anyone else noticed this problem? Should I go ahead and apply this
patch?

Best -- Eric

Footnotes:
[1]

>From 3c3f4ca9a34dca23051ca2f4e4518b416338d4f4 Mon Sep 17 00:00:00 2001
From: Eric Schulte <schulte.eric@gmail.com>
Date: Mon, 19 Jul 2010 13:45:25 -0700
Subject: [PATCH] org-table: fix infinite loop in table importation

* lisp/org-table.el (org-table-convert-region): don't continue csv
  importation which the point catches the end, this fixes an infinite
  loop which was caused by the (point) never catching up with the
  "end" marker
 		2010-07-20 09:28:22 +02:00
..
ob-C.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-R.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-asymptote.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-clojure.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-comint.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-css.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-ditaa.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-dot.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-emacs-lisp.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-eval.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-exp.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-gnuplot.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-haskell.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-keys.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-latex.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-lob.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-matlab.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-mscgen.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-ocaml.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-octave.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-perl.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-python.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-ref.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-ruby.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-sass.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-screen.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-sh.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-sql.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-sqlite.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-table.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob-tangle.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
ob.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-agenda.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-archive.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-ascii.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-attach.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-bbdb.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-beamer.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-bibtex.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-capture.el Make org-capture use `org-default-notes-file' if the file is not specified 2010-07-20 09:05:31 +02:00
org-clock.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-colview-xemacs.el Estimate ranges in column view 2010-07-19 13:56:17 +02:00
org-colview.el Estimate ranges in column view 2010-07-19 13:56:17 +02:00
org-compat.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-crypt.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-ctags.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-datetree.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-docbook.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-docview.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-entities.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-exp-blocks.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-exp.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-faces.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-feed.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-footnote.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-freemind.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-gnus.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-habit.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-html.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-icalendar.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-id.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-indent.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-info.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-inlinetask.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-irc.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-jsinfo.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-latex.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-list.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-mac-message.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-macs.el Make org-capture use `org-default-notes-file' if the file is not specified 2010-07-20 09:05:31 +02:00
org-mew.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-mhe.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-mks.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-mobile.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-mouse.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-plot.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-protocol.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-publish.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-remember.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-rmail.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-src.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-table.el fix infinite loop in csv table import 2010-07-20 09:28:22 +02:00
org-taskjuggler.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-timer.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-vm.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-w3m.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-wl.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org-xoxo.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00
org.el Mention release 7.01 on the website, push release number to 7.01trans 2010-07-19 08:33:24 +02:00