gcov improvements

git-svn-id: https://svn.o-hand.com/repos/matchbox/trunk/matchbox-tests@792 b067294f-1dea-0310-9683-c47a78595994

ChangeLog

@@ -1,3 +1,11 @@
+2004-04-28  Matthew Allum,,,  <mallum@handhelds.org>
+
+	* gcov-notes.txt:
+	Add some notes and stats on gcov usage
+	* test-interaction-240x320.xnee:
+	* test-interaction.sh:
+	Redo for matchbox-desktop usage and improved gcov stats
+
 2004-04-26  Matthew Allum,,,  <mallum@handhelds.org>
 
 	* gcov-wrapper.sh:

gcov-notes.txt

@@ -0,0 +1,61 @@
+Gcov Notes for 28/04/2004
+====
+
+Here are the current gcov-wrap.sh statistics for a 'vanilla' matchbox
+build running on a 240x320 Xnest resolution;
+
+( Refer to the matchbox test documentation for info on matchbox gcov setup )
+
+matchbox-window-manager gcov data:
+ 
+89.09% of 220 lines executed in file base_client.c
+79.89% of 184 lines executed in file client_common.c
+0.00% of 2 lines executed in file composite-engine.c
+91.18% of 34 lines executed in file desktop_client.c
+88.68% of 265 lines executed in file dialog_client.c
+82.18% of 101 lines executed in file dockbar_client.c
+69.91% of 226 lines executed in file ewmh.c
+84.85% of 198 lines executed in file keys.c
+78.38% of 37 lines executed in file list.c
+93.33% of 15 lines executed in file main.c
+77.90% of 267 lines executed in file main_client.c
+38.14% of 97 lines executed in file matchbox-remote.c
+Could not open basic block file mbtheme-standalone.bb.
+78.88% of 1032 lines executed in file mbtheme.c
+58.62% of 58 lines executed in file misc.c
+63.45% of 145 lines executed in file select_client.c
+94.77% of 172 lines executed in file toolbar_client.c
+61.62% of 779 lines executed in file wm.c
+84.21% of 228 lines executed in file xml.c
+ 
+libmatchbox gcov data:
+ 
+89.36% of 47 lines executed in file hash.c
+60.09% of 213 lines executed in file mbdotdesktop.c
+74.95% of 459 lines executed in file mbexp.c
+34.41% of 744 lines executed in file mbmenu.c
+65.00% of 860 lines executed in file mbpixbuf.c
+69.39% of 477 lines executed in file mbtray.c
+21.08% of 166 lines executed in file mbutil.c
+
+Notes;
+===
+
+- matchbox-remote.c can be ignored, its a seperate application to the window
+  manager - its just its source lives in the same directory as the wm so the
+  script picks it up.
+
+- One would expect wm.c percentages to be higher - Reasons for low values
+  ( via wm.c,gcov output ) are;
+
+  - No way to simulate a randr event ( eg a screen rotation ) in xnest.
+    A big chunk of code in wm.c is for handling this situation.
+
+  - Command line switches are handled here and the tests are run without
+    any. 
+  
+  - Some keyboard shortcuts and matchbox-remote messages are not used by
+    the tests. 
+
+
+- The mbtest theme could possibly be improved to increase mbtheme percentages

test-interaction.sh

@@ -55,14 +55,18 @@ fi
 matchbox-remote -t mbtest
 
 # Launch various window types. 
+(matchbox-desktop || echo "*** Unable to start matchbox-desktop - is it installed?") &
+
 ./winspew -np -t "Window",normal 2>&1 > /dev/null &
 ./winspew -np -t "Window",dock,+0+300 2>&1 > /dev/null &
 ./winspew -np -t "Window",toolbar 2>&1 > /dev/null &
-./winspew -np -t "Window",normal -c "Dialog",160x110 -c "Dialog",100x100 2>&1 > /dev/null &
-./winspew -np -t "Window",normal -c "Dialog",160x150,accept_button:help_button 2>&1 > /dev/null &
-./winspew -np -t "Window",normal -c "Dialog",320x110,no_decor 2>&1 > /dev/null &
 
-matchbox-desktop &
+## Add some sleeps to avoid race where final display order can differ...
+sleep 1
+./winspew -np -t "Window",normal -c "Dialog",160x110 -c "Dialog",100x100 2>&1 > /dev/null &
+sleep 1
+./winspew -np -t "Window",normal -c "Dialog",160x150,accept_button:help_button 2>&1 > /dev/null &
+
 
 # Now playback recorded interactions with Xnee. 
 #
@@ -110,6 +114,7 @@ function file_compare ()
 
 
 if [ -z "$WANT_IMG_COMPARE" ]; then
+   killall winspew matchbox-desktop
    exit 0
 fi