* Re: master e870279 1/2: Quick temporary hack to fix curcell refreshing.
[not found] ` <E1aEJq1-0006ep-OD@vcs.savannah.gnu.org>
@ 2015-12-30 17:38 ` Óscar Fuentes
0 siblings, 0 replies; 2+ messages in thread
From: Óscar Fuentes @ 2015-12-30 17:38 UTC (permalink / raw)
To: emacs-devel; +Cc: Vincent Belaïche
Please install bugfixes on the emacs-25 branch.
Your change addresses bug#21054, which is a release blocker.
Vincent Belache <vincentb1@users.sourceforge.net> writes:
> branch: master
> commit e8702794d46ae032803bf54ffbd71ebde215179c
> Author: Vincent Belaïche <vincentb1@users.sourceforge.net>
> Commit: Vincent Belaïche <vincentb1@users.sourceforge.net>
>
> Quick temporary hack to fix curcell refreshing.
>
> The problem was caused by change: 2015-04-13 Deprecate `intangible'
> and `point-entered' properties. The problem is that this change has
> removed the (setq ses--curcell t) setting in the ses-command-hook
> function.
>
> * ses.el (ses-check-curcell): replace `(eq ses--curcell t)' by just `t' as
> a condition to call function `ses-set-curcell'. Comment this as a quick
> temporary hack to make it work, as I don't know yet whether a definite
> correction would be to make the ses-set-curcell at every ses-check-curcell,
> or to revert to the previous approach, ie marking ses--curcell as out-of-date
> at every potentially cursor motion command.
> ---
> lisp/ses.el | 6 +++++-
> 1 files changed, 5 insertions(+), 1 deletions(-)
>
> diff --git a/lisp/ses.el b/lisp/ses.el
> index ec1359b..7347eb8 100644
> --- a/lisp/ses.el
> +++ b/lisp/ses.el
> @@ -1117,7 +1117,11 @@ region, or nil if cursor is not at a cell."
> The end marker is appropriate if some argument is 'end.
> A range is appropriate if some argument is 'range.
> A single cell is appropriate unless some argument is 'needrange."
> - (if (eq ses--curcell t)
> + (if t;; Vincent B.:
> + ;; previously (eq ses--curcell t)
> + ;; this is a quick hack to temporary fix the bug implied by changes made by Stefan in
> + ;; http://git.savannah.gnu.org/cgit/emacs.git/commit/lisp/ses.el?id=84e0b7dad6f1a8e53261f9b96f5a9080fea681a4
> + ;;-------------------------------------------------------------------------
> ;; curcell recalculation was postponed, but user typed ahead.
> (ses-set-curcell))
> (cond
>
> _______________________________________________
> Emacs-diffs mailing list
> Emacs-diffs@gnu.org
> https://lists.gnu.org/mailman/listinfo/emacs-diffs
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: master e870279 1/2: Quick temporary hack to fix curcell refreshing.
@ 2015-12-31 8:07 Vincent Belaïche
0 siblings, 0 replies; 2+ messages in thread
From: Vincent Belaïche @ 2015-12-31 8:07 UTC (permalink / raw)
To: Óscar Fuentes; +Cc: emacs-devel
Dear Óscar,
I will do that ASAP. I think that it is desirable also to make the same
correction as
http://git.savannah.gnu.org/cgit/emacs.git/commit/?id=0c9abf36a4fbc775918c08c1104a64d037280abc
titled "Correct ses-rename-cell cursor-intangible text prop updating."
That was a hidden bug that had remained unoticed to me up to
now. However, after the changes made by Stefan on 2015-04-13T19:51:15Z
(Deprecate `intangible' and `point-entered' properties), this became a
nuisance every time you rename a cell --- so maybe not so big a
nuisance, if you don't use cell renaming.
FYI, renamed cells still need some work to be fully functional (notably
for not relocating their name after some column/row insertion), but I
will do that on the master branch only.
VBR,
Vincent
Le 30/12/2015 18:38, Óscar Fuentes a écrit :
Please install bugfixes on the emacs-25 branch.
Your change addresses bug#21054, which is a release blocker.
Vincent Belache <vincentb1@users.sourceforge.net> writes:
branch: master
commit e8702794d46ae032803bf54ffbd71ebde215179c
Author: Vincent Belaïche <vincentb1@users.sourceforge.net>
Commit: Vincent Belaïche <vincentb1@users.sourceforge.net>
Quick temporary hack to fix curcell refreshing.
The problem was caused by change: 2015-04-13 Deprecate `intangible'
and `point-entered' properties. The problem is that this change has
removed the (setq ses--curcell t) setting in the ses-command-hook
function.
* ses.el (ses-check-curcell): replace `(eq ses--curcell t)' by just `t' as
a condition to call function `ses-set-curcell'. Comment this as a quick
temporary hack to make it work, as I don't know yet whether a definite
correction would be to make the ses-set-curcell at every ses-check-curcell,
or to revert to the previous approach, ie marking ses--curcell as out-of-date
at every potentially cursor motion command.
---
lisp/ses.el | 6 +++++-
1 files changed, 5 insertions(+), 1 deletions(-)
diff --git a/lisp/ses.el b/lisp/ses.el
index ec1359b..7347eb8 100644
--- a/lisp/ses.el
+++ b/lisp/ses.el
@@ -1117,7 +1117,11 @@ region, or nil if cursor is not at a cell."
The end marker is appropriate if some argument is 'end.
A range is appropriate if some argument is 'range.
A single cell is appropriate unless some argument is 'needrange."
- (if (eq ses--curcell t)
+ (if t;; Vincent B.:
+ ;; previously (eq ses--curcell t)
+ ;; this is a quick hack to temporary fix the bug implied by changes made by Stefan in
+ ;; http://git.savannah.gnu.org/cgit/emacs.git/commit/lisp/ses.el?id=84e0b7dad6f1a8e53261f9b96f5a9080fea681a4
+ ;;-------------------------------------------------------------------------
;; curcell recalculation was postponed, but user typed ahead.
(ses-set-curcell))
(cond
_______________________________________________
Emacs-diffs mailing list
Emacs-diffs@gnu.org
https://lists.gnu.org/mailman/listinfo/emacs-diffs
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-12-31 8:07 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20151230164301.25523.22477@vcs.savannah.gnu.org>
[not found] ` <E1aEJq1-0006ep-OD@vcs.savannah.gnu.org>
2015-12-30 17:38 ` master e870279 1/2: Quick temporary hack to fix curcell refreshing Óscar Fuentes
2015-12-31 8:07 Vincent Belaïche
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.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).