gnugo-devel
[Top][All Lists]
Advanced

[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 ()




reply via email to

[Prev in Thread] Current Thread [Next in Thread]