Move HandleWrap rules to one place

diff --git a/src/handle_wrap.h b/src/handle_wrap.h
index 2c51799a5885bdefec1f1ce51610d3c1b109b00b..8329da04170fd7d77057ec0c4a2fa9e348a5f99d 100644 (file)
--- a/src/handle_wrap.h
+++ b/src/handle_wrap.h
@@ -3,6 +3,26 @@
 
 namespace node {
 
+// Rules:
+//
+// - Do not throw from handle methods. Set errno.
+//
+// - MakeCallback may only be made directly off the event loop.
+//   That is there can be no JavaScript stack frames underneith it.
+//   (Is there anyway to assert that?)
+//
+// - No use of v8::WeakReferenceCallback. The close callback signifies that
+//   we're done with a handle - external resources can be freed.
+//
+// - Reusable?
+//
+// - The uv_close_cb is used to free the c++ object. The close callback
+//   is not made into javascript land.
+//
+// - uv_ref, uv_unref counts are managed at this layer to avoid needless
+//   js/c++ boundary crossing. At the javascript layer that should all be
+//   taken care of.
+
 class HandleWrap {
   public:
     static void Initialize(v8::Handle<v8::Object> target);

diff --git a/src/pipe_wrap.cc b/src/pipe_wrap.cc
index 4d8f35239e3e4ca6b7f61c38aba0775da603095e..adbeb0f8c18d0fd0583eae81b7b6ea4b985575b0 100644 (file)
--- a/src/pipe_wrap.cc
+++ b/src/pipe_wrap.cc
@@ -4,27 +4,6 @@
 #include <handle_wrap.h>
 #include <stream_wrap.h>
 
-// Rules:
-//
-// - Do not throw from handle methods. Set errno.
-//
-// - MakeCallback may only be made directly off the event loop.
-//   That is there can be no JavaScript stack frames underneith it.
-//   (Is there anyway to assert that?)
-//
-// - No use of v8::WeakReferenceCallback. The close callback signifies that
-//   we're done with a handle - external resources can be freed.
-//
-// - Reusable?
-//
-// - The uv_close_cb is used to free the c++ object. The close callback
-//   is not made into javascript land.
-//
-// - uv_ref, uv_unref counts are managed at this layer to avoid needless
-//   js/c++ boundary crossing. At the javascript layer that should all be
-//   taken care of.
-
-
 #define UNWRAP \
   assert(!args.Holder().IsEmpty()); \
   assert(args.Holder()->InternalFieldCount() > 0); \

diff --git a/src/tcp_wrap.cc b/src/tcp_wrap.cc
index fd4552c9701abae755a049469fdfa2bc317846fc..814f7e235b8eeaf821cb64443e89c738a283000b 100644 (file)
--- a/src/tcp_wrap.cc
+++ b/src/tcp_wrap.cc
@@ -19,27 +19,6 @@
 # define uv_inet_ntop inet_ntop
 #endif
 
-// Rules:
-//
-// - Do not throw from handle methods. Set errno.
-//
-// - MakeCallback may only be made directly off the event loop.
-//   That is there can be no JavaScript stack frames underneith it.
-//   (Is there anyway to assert that?)
-//
-// - No use of v8::WeakReferenceCallback. The close callback signifies that
-//   we're done with a handle - external resources can be freed.
-//
-// - Reusable?
-//
-// - The uv_close_cb is used to free the c++ object. The close callback
-//   is not made into javascript land.
-//
-// - uv_ref, uv_unref counts are managed at this layer to avoid needless
-//   js/c++ boundary crossing. At the javascript layer that should all be
-//   taken care of.
-
-
 #define UNWRAP \
   assert(!args.Holder().IsEmpty()); \
   assert(args.Holder()->InternalFieldCount() > 0); \

diff --git a/src/timer_wrap.cc b/src/timer_wrap.cc
index 3d45946ce6df56b4d1da03b7026047f31425812b..5e1fd25a54323129672df9555bcbf1cf3dc7df08 100644 (file)
--- a/src/timer_wrap.cc
+++ b/src/timer_wrap.cc
@@ -1,27 +1,6 @@
 #include <node.h>
 #include <handle_wrap.h>
 
-// Rules:
-//
-// - Do not throw from handle methods. Set errno.
-//
-// - MakeCallback may only be made directly off the event loop.
-//   That is there can be no JavaScript stack frames underneith it.
-//   (Is there anyway to assert that?)
-//
-// - No use of v8::WeakReferenceCallback. The close callback signifies that
-//   we're done with a handle - external resources can be freed.
-//
-// - Reusable?
-//
-// - The uv_close_cb is used to free the c++ object. The close callback
-//   is not made into javascript land.
-//
-// - uv_ref, uv_unref counts are managed at this layer to avoid needless
-//   js/c++ boundary crossing. At the javascript layer that should all be
-//   taken care of.
-
-
 #define UNWRAP \
   assert(!args.Holder().IsEmpty()); \
   assert(args.Holder()->InternalFieldCount() > 0); \