Changeset 110 for branches

Timestamp:

12/11/07 03:28:39 (13 months ago)

Author:

follower

Message:

If the connection 'listen' method fails we bail into an infinite loop. TODO: Provide a way for the caller to detect and handle this kind of error?

Files:

• branches/follower/wiz810mj/src/demo/WizDemo4/WizDemo4.pde (modified) (3 diffs)

branches/follower/wiz810mj/src/demo/WizDemo4/WizDemo4.pde

@@ -218 +218 @@
 /* ---- NetworkConnection ---- */
 
-
 // TODO: Make this 'NetworkServerConnection'? Or just 'ServerConnection'?
 // TODO: Pull one-line methods into class definition to allow inlining?
@@ -283 +282 @@
 
 /* --  NetworkInterface -- */
+
+#define HANDLE_BAD_ERROR() while (1) {};
 
 // TODO?: Do we want to have a "default" socket accessible by 'Network.read()' etc?
@@ -335 +336 @@
     NetworkConnection connection = NetworkConnection(port);
     
-    connection.listen();
+    // TODO: How to best handle errors?
+    //       There's two major categories I can think of currently:
+    //         1) A socket wasn't available to listen in the first place
+    //         2) The listen attempt failed for some reason
+    //       Using 'HANDLE_BAD_ERROR()' (currently an infinite loop) is
+    //       not ideal but should prevent the above errors slipping by unnoticed.
+    if (!connection.listen()) {
+      HANDLE_BAD_ERROR(); 
+    }
     
     return connection;