Re: [O] [Bug] Regression w/ insert into tables in overwrite-mode

2012-03-27 Thread Bastien
Hi Achim,

Achim Gratz strom...@nexgo.de writes:

 Achim Gratz strom...@nexgo.de writes:

 Commit 00040e7 by David Maus has introduced a regression when inserting
 into org tables while overwrite mode is on.  For each character inserted
 into a table cell, the cell (and the whole table row) gets shortened by
 one character.

 If the change can not be reverted, the deletion must not
 happen when overwrite mode is on, maybe through something like this:

   (if (not overwrite-mode) (delete-char -1))

 The change cannot be reverted, since Emacs24 issues a compile-time
 warning about using delete-backward-char, even though that warning has
 been dropped from the doc string... so it needs to be protected or
 replaced by an org-specific version that honors overwrite mode.

I reverted David's change and replaced `delete-backward-char'
by `backward-delete-char', which isn't caught by the compiler*.

`backward-delete-char' is remapped to `delete-backward-char' 
by Emacs and `delete-backward-char' is remapped by Org to
`org-delete-backward-char'.

Thanks for your patience on this one!

* This looks weird that an alias isn't caught by the compiler, 
I'll let Emacs devs know about this.

-- 
 Bastien



Re: [O] [Bug] Regression w/ insert into tables in overwrite-mode

2012-03-16 Thread Achim Gratz
Achim Gratz strom...@nexgo.de writes:
 Commit 00040e7 by David Maus has introduced a regression when inserting
 into org tables while overwrite mode is on.  For each character inserted
 into a table cell, the cell (and the whole table row) gets shortened by
 one character.

 If the change can not be reverted, the deletion must not
 happen when overwrite mode is on, maybe through something like this:

   (if (not overwrite-mode) (delete-char -1))

 Emacs24 actually does not have the doc string that admonishes not to use
 the function delete-backward-char in non-interactive mode, so the point
 of the commit may actually be moot (although neither the doc string of
 delete-char nor delete-backward char make much mention of overwrite
 mode).  Interestingly, even though I am inside the table,
 org-self-insert-command gets called and not orgtbl-self-insert-command,
 so I can't really say if both changes would need to be fixed in the same
 way.

 Additionally, it appears that the implementation and the doc string for
 orgtbl-self-insert-command don't match, there doesn't seem to be any use
 of overwrite mode in its implementation.

*Bump*


Regards,
Achim.
-- 
+[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]+

Factory and User Sound Singles for Waldorf Q+, Q and microQ:
http://Synth.Stromeko.net/Downloads.html#WaldorfSounds




Re: [O] [Bug] Regression w/ insert into tables in overwrite-mode

2012-03-05 Thread Achim Gratz
Achim Gratz strom...@nexgo.de writes:
 Commit 00040e7 by David Maus has introduced a regression when inserting
 into org tables while overwrite mode is on.  For each character inserted
 into a table cell, the cell (and the whole table row) gets shortened by
 one character.

*Ping*

David, are you listening?
:-)


Regards,
Achim.
-- 
+[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]+

Wavetables for the Terratec KOMPLEXER:
http://Synth.Stromeko.net/Downloads.html#KomplexerWaves




Re: [O] [Bug] Regression w/ insert into tables in overwrite-mode

2012-02-27 Thread Achim Gratz
Achim Gratz strom...@nexgo.de writes:

 Commit 00040e7 by David Maus has introduced a regression when inserting
 into org tables while overwrite mode is on.  For each character inserted
 into a table cell, the cell (and the whole table row) gets shortened by
 one character.

 If the change can not be reverted, the deletion must not
 happen when overwrite mode is on, maybe through something like this:

   (if (not overwrite-mode) (delete-char -1))

The change cannot be reverted, since Emacs24 issues a compile-time
warning about using delete-backward-char, even though that warning has
been dropped from the doc string... so it needs to be protected or
replaced by an org-specific version that honors overwrite mode.


Regards,
Achim.
-- 
+[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]+

Samples for the Waldorf Blofeld:
http://Synth.Stromeko.net/Downloads.html#BlofeldSamplesExtra