Small suggestion and question

Small suggestion and question

[leonp]

Hello,
         I use the SN for cross development (this time under Win2K without 
CygWin).

1 - Suggestion:
         It seems to be comfortable to have the "build" window be open as a 
subwindow in text editor - it is always accessible and one can rebuild his 
target fast after any small changes.      What seems to be not so 
comfortable is:
         a) The necessity to confirm each change before build starts (may 
be by default, if I pressed "build" button - save).
         b) This "build" subwindow seems to occupy too much place with the 
information which isn't so necessary at the moment - build command, build 
directory, target. If you ask me, the result window with 3 buttons at the 
same horizontal level will be brilliant.

2 - Question:
         When I run SN under Win2K it is VERY slow, which, besides making 
the work nervous, causes SN errors. For example, pressing Ctrl-S and Ctrl-W 
sequentually, causes an error below. This effect doesn't occur when running 
under Linux on the same machine. Does somebody know any medicine to the 
problem?

Error:
wrong # ".multiwindow-2.nbook.nbframe.edit.paned.edit.edit.edit" does not exist
-----------------------------------------------------------
Comment:
-----------------------------------------------------------
Version:        4.5.2
-----------------------------------------------------------
Full Name:      Red Hat Embedded DevKit
-----------------------------------------------------------
Product Name:   :Source-Navigator
-----------------------------------------------------------
sn_home:        c:/program files/cygnus/sn452
pwd:            C:/AGM142CP/SOURCES
project_dir:    C:/AGM142CP/SOURCES
project_file:   C:/AGM142CP/SOURCES/ofp.proj
db created with SN version: 4.5.2
db project version: 41001
db_prefix:      .snprj/ofp
db-directory:   .snprj  mode: 040744    directory
.snprj/ofp.f:   mode: 0100644   size: 2048
.snprj/ofp.fu:  mode: 0100644   size: 4608
.snprj/ofp.fil: mode: 0100644   size: 37376
.snprj/ofp.iu:  mode: 0100644   size: 2560
.snprj/ofp.ma:  mode: 0100644   size: 14336
.snprj/ofp.gv:  mode: 0100644   size: 4096
.snprj/ofp.iv:  mode: 0100644   size: 7680
.snprj/ofp.cl:  mode: 0100644   size: 1024
.snprj/ofp.ctr: mode: 0100644   size: 0
.snprj/ofp.to:  mode: 0100644   size: 71168
.snprj/ofp.by:  mode: 0100644   size: 73216
.snprj/ofp.un:  mode: 0100644   size: 1024
.snprj/ofp.fd:  mode: 0100644   size: 3584
-----------------------------------------------------------
osVersion: 5.0 byteOrder: littleEndian machine: intel platform: windows os: 
Windows NT
-----------------------------------------------------------
Stack:
wrong # ".multiwindow-2.nbook.nbframe.edit.paned.edit.edit.edit" does not exist
     while invoking
"Sn_Syntax_Highlight -delall c++ 
.multiwindow-2.nbook.nbframe.edit.paned.edit.ed..."
     invoked from within
"Sn_Syntax_Highlight -delall $grp $editor 1.0 end"
     (object ".multiwindow-2.nbook.nbframe.edit.paned." method 
"Editor&@@UpdateHighlights" body line 35)
     invoked from within
"UpdateHighlights $run_parse"
     (object ".multiwindow-2.nbook.nbframe.edit.paned." method 
"Editor&@@color_file" body line 3)
     invoked from within
"color_file"
     (object ".multiwindow-2.nbook.nbframe.edit.paned." method 
"Editor&@@save_file" body line 56)
     invoked from within
".multiwindow-2.nbook.nbframe.edit.paned.edit.edit save_file"
     (command bound to event)




Thanks a lot.


Leon Pollak
leonp@plris.com

Re: Small suggestion and question

[Mo DeJong]

On Sun, 1 Apr 2001 leonp@plris.com wrote:

> Hello,
>          I use the SN for cross development (this time under Win2K without 
> CygWin).
> 
> 1 - Suggestion:
>          It seems to be comfortable to have the "build" window be open as a 
> subwindow in text editor - it is always accessible and one can rebuild his 
> target fast after any small changes.      What seems to be not so 
> comfortable is:
>          a) The necessity to confirm each change before build starts (may 
> be by default, if I pressed "build" button - save).

Sounds interesting, basically a "fast save all" by default.
A prefs option for that would not be too hard to add.

>          b) This "build" subwindow seems to occupy too much place with the 
> information which isn't so necessary at the moment - build command, build 
> directory, target. If you ask me, the result window with 3 buttons at the 
> same horizontal level will be brilliant.

I think we would like to merge the dir and the command into
the target itself. That way you could have targets like:

autoconf: run ./configure
make: run make

You could also have targets that just call other
targets, like autoconf + make. A little bit of
work on the targets piece of the tools could
make SN a whole lot more useful. You
could do neat stuff like kick off a simulator.

> 2 - Question:
>          When I run SN under Win2K it is VERY slow, which, besides making 
> the work nervous, causes SN errors. For example, pressing Ctrl-S and Ctrl-W 
> sequentually, causes an error below. This effect doesn't occur when running 
> under Linux on the same machine. Does somebody know any medicine to the 
> problem?

I think that problem was fixed in the soon to be released 5.0
version. The attached patch should fix it for you in 4.5.

Mo
Index: busy.tcl
===================================================================
RCS file: /cvs/cvsfiles/devo/snavigator/gui/Attic/busy.tcl,v
retrieving revision 1.6.10.2
retrieving revision 1.6.10.2.14.1
diff -u -r1.6.10.2 -r1.6.10.2.14.1
--- busy.tcl	2000/02/11 23:48:55	1.6.10.2
+++ busy.tcl	2000/12/14 00:23:46	1.6.10.2.14.1
@@ -215,7 +215,7 @@
 		#already called
 		if {! [info exists watch_num_calls($win)]} {
 			sn_blt_busy hold_update $win
-			set watch_num_calls($win) 1
+			set watch_num_calls($win) {}
 		} else {
 			sn_log "WARNING: (watch y $win) already called"; flush stderr
 		}
@@ -226,10 +226,35 @@
 				update idletasks;		# Important !
 			}
 			sn_blt_busy release $win
+			
+			# Run any watch_done callbacks
+			if {$watch_num_calls($win) != {}} {
+			    foreach cmd $watch_num_calls($win) {
+			        sn_log "running watch $win release command \"$cmd\""
+			        uplevel #0 $cmd
+			    }
+			}
 			catch {unset watch_num_calls($win)}
 		} else {
 			sn_log "WARNING: (watch n $win) wrongly called"; flush stderr
 		}
+	}
+}
+
+# This procedure will run the given cmd when the watch for a given window
+# is released. If the given window is not currently locked with a watch
+# then the command will be run right away.
+
+proc watch_done { win cmd } {
+	global watch_num_calls
+	if {! [info exists watch_num_calls($win)]} {
+	    # Run the command right away
+	    sn_log "!!!!!!!!!!!!!! Watch NOT held for $win, executing \"$cmd\""
+	    uplevel #0 $cmd
+	} else {
+	    # Save it so that it will get run when the watch is released
+	    sn_log "!!!!!!!!!!!!!! Watch HELD for $win release command \"$cmd\""
+	    lappend watch_num_calls($win) $cmd
 	}
 }
 
Index: multiappl.tcl
===================================================================
RCS file: /cvs/cvsfiles/devo/snavigator/gui/Attic/multiappl.tcl,v
retrieving revision 1.32.2.17.2.5.4.7
retrieving revision 1.32.2.17.2.5.4.7.14.1
diff -u -r1.32.2.17.2.5.4.7 -r1.32.2.17.2.5.4.7.14.1
--- multiappl.tcl	2000/02/11 23:49:20	1.32.2.17.2.5.4.7
+++ multiappl.tcl	2000/12/14 00:23:46	1.32.2.17.2.5.4.7.14.1
@@ -4099,7 +4099,11 @@
 	#this procedure is bound on Control-w, so don't
 	#close window when it's the last one in the project
 	if {[MultiWindow& @@ num_Existing_windows] > 1} {
-		catch {[winfo toplevel $w] windows_close dummy}
+	        # Protect against destruction of the toplevel
+		# while the window is locked by a call to
+		# [watch $top y]. If there is no watch
+	        set top [winfo toplevel $w]
+		watch_done $top "$top windows_close dummy"
 	}
 }
 proc find_dialog {w} {