Changeset 164

Timestamp:

03/19/09 13:37:36 (15 years ago)

Author:

bhilburn

Message:

Added some TODOs to the following files:

Location:

vtcross/trunk/src

Files:

• lib/socketcomm/socketcomm.cpp (modified) (1 diff)
• policy_engines/PolicyEngine.cpp (modified) (1 diff)
• service_management_layer/ServiceManagementLayer.cpp (modified) (4 diffs)

vtcross/trunk/src/lib/socketcomm/socketcomm.cpp

@@ -23 +23 @@
 
 
+// TODO can someone write a description of how this function is operating? I'm
+// not sure I understand why it is making two separate blocking calls to recv
 void
 ReadMessage(int32_t socketFD, char* msgBuffer)

vtcross/trunk/src/policy_engines/PolicyEngine.cpp

@@ -47 +47 @@
     struct Parameter pList[10];
     struct CE_Info ce_info;
-    int32_t ret = GetRequest(socketFD, pList, &ce_info);
-   
-    if(ret == 1) {
+
+    // TODO should this be a GetRequest call or something else?
+    if(GetRequest(socketFD, pList, &ce_info)) {
         LOG("Policy Engine:: Validating Transmission Parameters.\n");
         ValidateParameters(pList, &ce_info, decision_array);    

vtcross/trunk/src/service_management_layer/ServiceManagementLayer.cpp

@@ -23 +23 @@
 ServiceManagementLayer::ServiceManagementLayer()
 {
+    LOG("Creating Service Management Layer.\n");
+    LoadConfiguration();
 }
 
@@ -39 +41 @@
 void 
 ServiceManagementLayer::WaitForSignal(int32_t socketFD)
-{
+{   
+    LOG("ServiceManagementLayer:: Waiting for signal...\n");
+
+    // TODO This is the critical point of this class... we need to figure out
+    // what the requests will look like before we can proceed.
+    while(1) {}
+
+    /*
+    if(GetRequest(socketFD, pList, &ce_info)) {
+        LOG("ServiceManagementLayer:: Received signal.\n");
+    }
+    */
+
 }
 
@@ -58 +72 @@
 ServiceManagementLayer::RegisterComponent(int32_t socketFD)
 {
+    SendMessage(socketFD, "sml_register");
+    LOG("ServiceManagementLayer:: Registration message sent.\n");
 }
 
@@ -106 +122 @@
 ServiceManagementLayer::LoadConfiguration()
 {
+    LOG("ServiceManagementLayer:: Loading Configuration.\n");
 }