gEDA-cvs: gaf.git: branch: master updated (1.6.1-20100214-116-g8a290f3)

[git@xxxxxxxxxx]

The branch, master has been updated
       via  8a290f30e3f89e7d2c7dd4459c02fa0dce366e02 (commit)
      from  b5cf93429f13cf9e68cc9cf86a05d72eae239f6f (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


=========
 Summary
=========

 gschem/src/o_move.c |   54 ++++++++++++++++++--------------------------------
 1 files changed, 20 insertions(+), 34 deletions(-)


=================
 Commit Messages
=================

commit 8a290f30e3f89e7d2c7dd4459c02fa0dce366e02
Author: Peter Clifton <pcjc2@xxxxxxxxx>
Commit: Peter Clifton <pcjc2@xxxxxxxxx>

    gschem: Tidy up o_move_prep_rubberband()
    
    Delete some debug printf and generally tidy up the function. Should
    be no functional changes.

:100644 100644 93f53c1... 6043f2f... M	gschem/src/o_move.c

=========
 Changes
=========

commit 8a290f30e3f89e7d2c7dd4459c02fa0dce366e02
Author: Peter Clifton <pcjc2@xxxxxxxxx>
Commit: Peter Clifton <pcjc2@xxxxxxxxx>

    gschem: Tidy up o_move_prep_rubberband()
    
    Delete some debug printf and generally tidy up the function. Should
    be no functional changes.

diff --git a/gschem/src/o_move.c b/gschem/src/o_move.c
index 93f53c1..6043f2f 100644
--- a/gschem/src/o_move.c
+++ b/gschem/src/o_move.c
@@ -540,47 +540,33 @@ void o_move_prep_rubberband(GSCHEM_TOPLEVEL *w_current)
   OBJECT *o_current;
   GList *iter;
 
-#if DEBUG
-  printf("\n\n\n");
-  s_stretch_print_all (toplevel->page_current->stretch_list);
-  printf("\n\n\n");
-#endif
+  for (s_current = geda_list_get_glist (toplevel->page_current->selection_list);
+       s_current != NULL; s_current = g_list_next (s_current)) {
+    object = s_current->data;
 
-  s_current = geda_list_get_glist( toplevel->page_current->selection_list );
-  while (s_current != NULL) {
-    object = (OBJECT *) s_current->data;
-    if (object) {
-      switch (object->type) {
-        case (OBJ_NET):
-        case (OBJ_PIN):
-        case (OBJ_BUS):
-          o_move_check_endpoint(w_current, object);
-          break;
+    if (object == NULL)
+      continue;
 
-        case (OBJ_COMPLEX):
-        case (OBJ_PLACEHOLDER):
-          iter = object->complex->prim_objs;
-          while (iter != NULL) {
-            o_current = (OBJECT *)iter->data;
+    switch (object->type) {
+      case (OBJ_NET):
+      case (OBJ_PIN):
+      case (OBJ_BUS):
+        o_move_check_endpoint (w_current, object);
+        break;
 
-            if (o_current->type == OBJ_PIN) {
-              o_move_check_endpoint(w_current, o_current);
-            }
+      case (OBJ_COMPLEX):
+      case (OBJ_PLACEHOLDER):
+        for (iter = object->complex->prim_objs;
+             iter != NULL; iter = g_list_next (iter)) {
+          o_current = iter->data;
 
-            iter = g_list_next (iter);
+          if (o_current->type == OBJ_PIN) {
+            o_move_check_endpoint (w_current, o_current);
           }
-
-          break;
-
-      }
+        }
+        break;
     }
-    s_current = g_list_next(s_current);
   }
-
-#if DEBUG
-  printf("\n\n\n\nfinished building scretch list:\n");
-  s_stretch_print_all (toplevel->page_current->stretch_list);
-#endif
 }
 
 /*! \todo Finish function documentation!!!




_______________________________________________
geda-cvs mailing list
geda-cvs@xxxxxxxxxxxxxx
http://www.seul.org/cgi-bin/mailman/listinfo/geda-cvs