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