Re: [O] [Bug] Regression w/ insert into tables in overwrite-mode
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
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
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
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
[O] [Bug] Regression w/ insert into tables in overwrite-mode
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. 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