From mboxrd@z Thu Jan 1 00:00:00 1970 From: Le Wang Subject: Re: C-0 C-c | throws emacs into infinite loop Date: Sat, 19 Mar 2011 18:03:08 +0800 Message-ID: References: Mime-Version: 1.0 Content-Type: multipart/alternative; boundary=20cf3054ab85602e00049ed30045 Return-path: Received: from [140.186.70.92] (port=44285 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Q0t06-0007RO-9M for emacs-orgmode@gnu.org; Sat, 19 Mar 2011 06:03:16 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Q0t01-0003uV-4C for emacs-orgmode@gnu.org; Sat, 19 Mar 2011 06:03:14 -0400 Received: from mail-vw0-f41.google.com ([209.85.212.41]:63855) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Q0t01-0003uJ-0E for emacs-orgmode@gnu.org; Sat, 19 Mar 2011 06:03:09 -0400 Received: by vws4 with SMTP id 4so5179785vws.0 for ; Sat, 19 Mar 2011 03:03:08 -0700 (PDT) In-Reply-To: List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org To: Carsten Dominik Cc: Orgmode Mailing List --20cf3054ab85602e00049ed30045 Content-Type: text/plain; charset=ISO-8859-1 On Sat, Mar 19, 2011 at 2:43 PM, Carsten Dominik wrote: > > On 18.3.2011, at 15:29, Le Wang wrote: > > > The doc-string of `org-table-convert-region' doesn't specifically address > 0 as SEPARATOR, but 0 is an integer. It shouldn't hang in any case. When I > C-g out of the loop, my undo limit was exceeded because the line filled with > hundreds of thousands of empty cells. > > > > I think the correct behavior is to convert region into table using 0 > spaces as a separator -- so each character forms its own column. > > While this might be nice, it is such a border case that, for now, I have > only added an > error message to prevent the converter from going into an infinite loop. > > Or do you have a *really* goo use case for this? > No, I don't have a use case for this. IMO, a reasonable argument should result in some reasonable non-surprising behaviour. We can argue whether 0 is a reasonable argument, though. I'm okay as long as it doesn't hang. > - Carsten > > -- Le --20cf3054ab85602e00049ed30045 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable
On Sat, Mar 19, 2011 at 2:43 PM, Carsten Dominik= <carsten.dominik@gmail.com> wrote:

On 18.3.2011, at 15:29, Le Wang wrote:

> The doc-string of `org-table-convert-region' doesn't specifica= lly address 0 as SEPARATOR, but 0 is an integer. =A0It shouldn't hang i= n any case. =A0When I C-g out of the loop, my undo limit was exceeded becau= se the line filled with hundreds of thousands of empty cells.
>
> I think the correct behavior is to convert region into table using 0 s= paces as a separator -- so each character forms its own column.

While this might be nice, it is such a border case that, for now, I h= ave only added an
error message to prevent the converter from going into an infinite loop.
Or do you have a *really* goo use case for this?

No, I don't have a use case for this.=A0 = IMO, a reasonable argument should result in some reasonable non-surprising = behaviour.=A0 We can argue whether 0 is a reasonable argument, though.=A0 I= 'm okay as long as it doesn't hang.

=A0
- Carsten




--
Le
--20cf3054ab85602e00049ed30045--