emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
* 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 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).