C-0 C-c | throws emacs into infinite loop

C-0 C-c | throws emacs into infinite loop

[Le Wang]

[-- Attachment #1: Type: text/plain, Size: 453 bytes --]

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.



-- 
Le


Org version 7.5
Emacs 24 Jan 2011 build

[-- Attachment #2: Type: text/html, Size: 521 bytes --]

Re: C-0 C-c | throws emacs into infinite loop

[Carsten Dominik]

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?

- Carsten

Re: C-0 C-c | throws emacs into infinite loop

[Le Wang]

[-- Attachment #1: Type: text/plain, Size: 1022 bytes --]

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 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

[-- Attachment #2: Type: text/html, Size: 1609 bytes --]