[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [gnugo-devel] Merge of gnugo.el patches
From: |
bump |
Subject: |
Re: [gnugo-devel] Merge of gnugo.el patches |
Date: |
Sun, 28 Nov 2004 08:23:01 -0800 |
Here's a small bugfix to the patch at:
http://lists.gnu.org/archive/html/gnugo-devel/2004-11/msg00099.html
This goes on top of the previous patch.
I wrote:
> To point out one problem with the patch, I found the
> function gnugo-merge-showboard-results difficult to
> understand and I didn't try to fix it. Instead I took out
> calls to this function and added a call to (gnugo-refresh
> t) at the end of gnugo-push-move. This works but it may
> be the cause of some slowness. After making a move there
> is a tiny delay (maybe 1/4 second on my machine, just
> enough to notice) before the move appears on the screen.
> So probably gnugo-merge-showboard-results is faster
> than calling gnugo-refresh and this needs to be fixed.
I no longer see this problem. Maybe the slowdown was
caused by some functions being instrumented for edebug.
Dan
Index: gnugo.el
===================================================================
RCS file: /home/bump/cvsroot/gnugoels/gnugo.el,v
retrieving revision 1.3.2.13
diff -u -r1.3.2.13 gnugo.el
--- gnugo.el 28 Nov 2004 02:38:47 -0000 1.3.2.13
+++ gnugo.el 28 Nov 2004 05:07:37 -0000
@@ -1496,7 +1498,7 @@
(defun gnugo-jump-to-end ()
"jump to the end of the game"
(interactive)
- (gnugo-jump-to-move (+ (length (gnugo-get :move-history))
+ (gnugo-jump-to-move (+ (length (cdr (gnugo-get :move-tree)))
(length (gnugo-get :future-history)))))
(defun gnugo-display-final-score ()