From 3497aabf5c1457a0c57ad5ca1f944af80e9f081e Mon Sep 17 00:00:00 2001
From: werner <werner@99fdad57-331a-0410-800a-d7fa5415bdb3>
Date: Wed, 26 Aug 2009 14:06:26 +0000
Subject: [PATCH] - destroy_obj: when deleting width expressions, check that
 they're actually   non-NULL - gui.html: added more details on selecting

git-svn-id: http://svn.openmoko.org/trunk/eda/fped@5542 99fdad57-331a-0410-800a-d7fa5415bdb3
---
 delete.c |  9 ++++++---
 gui.html | 16 ++++++++++++++++
 2 files changed, 22 insertions(+), 3 deletions(-)

diff --git a/delete.c b/delete.c
index d203c96..10cc285 100644
--- a/delete.c
+++ b/delete.c
@@ -211,13 +211,16 @@ static void destroy_obj(struct obj *obj)
 		free(obj->u.pad.name);
 		break;
 	case ot_line:
-		free_expr(obj->u.line.width);
+		if (obj->u.line.width)
+			free_expr(obj->u.line.width);
 		break;
 	case ot_rect:
-		free_expr(obj->u.rect.width);
+		if (obj->u.rect.width)
+			free_expr(obj->u.rect.width);
 		break;
 	case ot_arc:
-		free_expr(obj->u.arc.width);
+		if (obj->u.arc.width)
+			free_expr(obj->u.arc.width);
 		break;
 	case ot_meas:
 		if (obj->u.meas.label)
diff --git a/gui.html b/gui.html
index 46de6ea..a4a8ed2 100644
--- a/gui.html
+++ b/gui.html
@@ -143,6 +143,22 @@ Tools are used to add new elements and to manipulate existing ones.
   <DD> The pointer. This is the default tool. The pointer is used to
     select items and do move points of the selected item.
     <P>
+    Clicking on an item selects it. If items overlap, the one with the
+    highest priority is selected. The priority is based on how difficult
+    it usually is to select an item, with frame references having a low
+    priority, pads, circles, arcs, rectangles, measurements, and lines
+    having increasingly higher priorities. There are a few special cases:
+    <UL>
+      <LI> The circle at the end of a vector has the highest priority
+	while its line has the lowest priority.
+      <LI> To select a frame reference, click on the L-shaped upper left
+        corner.
+      <LI> To select a measurement, click on the line with the text, not
+	the (hypothetical) line connecting the points being measured.
+    </UL>
+    If multiple items are under the mouse pointer, repeatedly clicking
+    iterates through them.
+    <P>
     To move points, select the item, then move the mouse pointer over
     the point to move. A red circle will appear under the mouse pointer.
     Then drag the point to its new location and release the mouse button.