* C-0 C-c | throws emacs into infinite loop
@ 2011-03-18 14:29 Le Wang
2011-03-19 6:43 ` Carsten Dominik
0 siblings, 1 reply; 3+ messages in thread
From: Le Wang @ 2011-03-18 14:29 UTC (permalink / raw)
To: Orgmode Mailing List
[-- 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 --]
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: C-0 C-c | throws emacs into infinite loop
2011-03-18 14:29 C-0 C-c | throws emacs into infinite loop Le Wang
@ 2011-03-19 6:43 ` Carsten Dominik
2011-03-19 10:03 ` Le Wang
0 siblings, 1 reply; 3+ messages in thread
From: Carsten Dominik @ 2011-03-19 6:43 UTC (permalink / raw)
To: Le Wang; +Cc: Orgmode Mailing List
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
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: C-0 C-c | throws emacs into infinite loop
2011-03-19 6:43 ` Carsten Dominik
@ 2011-03-19 10:03 ` Le Wang
0 siblings, 0 replies; 3+ messages in thread
From: Le Wang @ 2011-03-19 10:03 UTC (permalink / raw)
To: Carsten Dominik; +Cc: Orgmode Mailing List
[-- 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 --]
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-03-19 10:03 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-03-18 14:29 C-0 C-c | throws emacs into infinite loop Le Wang
2011-03-19 6:43 ` Carsten Dominik
2011-03-19 10:03 ` Le Wang
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).