Removing some test calls

[staging/windowmanager.git] / AFBClient.cpp

diff --git a/AFBClient.cpp b/AFBClient.cpp
index 3d5a878..ece6501 100644 (file)
--- a/AFBClient.cpp
+++ b/AFBClient.cpp
@@ -33,7 +33,7 @@ AFBClient::~AFBClient()
 
 bool AFBClient::init()
 {
-    printf("init() -->\n");
+    printf("AFBClient::init() -->\n");
     /* get the default event loop */
     int rc = sd_event_default(&loop);
     if (rc < 0) {
@@ -48,43 +48,71 @@ bool AFBClient::init()
         return false;
     }
 
-    printf("init() <--\n");
+    printf("AFBClient::init() <--\n");
     return true;
 }
 
-void AFBClient::requestSurface(const char *label)
+int AFBClient::requestSurface(const char *label)
 {
-    printf("requestSurface(%s) -->\n", label);
-    fflush(stdout);
-
-    static int num = 0;
-    char *key;
-    int rc;
-    const char begin[] = "{\"drawing_name\":\"";
-    const char end[] = "\"}";
-    const char verb[] = "request_surface";
-    char *parameter = (char *)malloc(strlen(begin) +
-                                     strlen(label) +
-                                     strlen(end) + 1);
-    strcpy(parameter, begin);
-    strcat(parameter, label);
-    strcat(parameter, end);
-
-    /* allocates an id for the request */
-    rc = asprintf(&key, "%d:%s/%s", ++num, AFBClient::wmAPI, verb);
-
-    /* send the request */
-    rc = afb_wsj1_call_s(wsj1, AFBClient::wmAPI, verb, parameter, AFBClient::onRequestSurfaceReply, key);
-    if (rc < 0)
-        fprintf(stderr, "calling %s/%s(%s) failed: %m\n", AFBClient::wmAPI, verb, parameter);
-
-    printf("requestSurface(%s) <--\n", label);
-    fflush(stdout);
+   printf("AFBClient::requestSurface(%s) -->\n", label);
+   constexpr char const *verb = "request_surface";
+   int ret = -1;
+
+   json_object *jp = json_object_new_object();
+   json_object_object_add(jp, "drawing_name", json_object_new_string(label));
+
+   // std::experimental::optional look-alike
+   struct optional {
+      int value;
+      bool is_not_set;
+   };
+
+   constexpr struct optional const nullopt = {0, true};
+   auto id = nullopt;
+
+   /* send the request */
+   int rc = afb_wsj1_call_j(
+      wsj1, AFBClient::wmAPI, verb, jp,
+      [](void *closure, afb_wsj1_msg *msg) {
+         if (afb_wsj1_msg_is_reply_ok(msg)) {
+            int id = json_object_get_int(
+               json_object_object_get(afb_wsj1_msg_object_j(msg), "response"));
+            auto oid = (optional *)closure;
+            *oid = optional{id};
+         } else
+            fprintf(stderr, "wrong request surface reply received!\n");
+      },
+      (void *)&id);
+
+   if (rc < 0) {
+      fprintf(stderr, "calling %s/%s(%s) failed: %m\n", AFBClient::wmAPI, verb,
+              json_object_to_json_string(jp));
+   } else {
+      // Lets make this call sync here...
+      dispatch(-1);
+
+      if (! id.is_not_set) {
+          char *buf;
+          asprintf(&buf, "%d", id.value);
+          printf("setenv(\"QT_IVI_SURFACE_ID\", %s, 1)\n", buf);
+          if (setenv("QT_IVI_SURFACE_ID", buf, 1) != 0) {
+              fprintf(stderr, "putenv failed: %m\n");
+          } else {
+              ret = 0; // Single point of success
+          }
+      } else {
+          fprintf(stderr, "Could not get surface ID from WM\n");
+      }
+   }
+
+   printf("AFBClient::requestSurface(%s) = %d <--\n", label, ret);
+
+   return ret;
 }
 
 void AFBClient::activateSurface(const char *label)
 {
-    printf("activateSurface(%s) -->\n", label);
+    printf("AFBClient::activateSurface(%s) -->\n", label);
     fflush(stdout);
 
     const char begin[] = "{\"drawing_name\":\"";
@@ -98,22 +126,41 @@ void AFBClient::activateSurface(const char *label)
     strcat(parameter, end);
     call(AFBClient::wmAPI, verb, parameter);
 
-    printf("activateSurface(%s) <--\n", label);
+    // Sync this one too
+    dispatch(-1);
+
+    printf("AFBClient::activateSurface(%s) <--\n", label);
     fflush(stdout);
 }
 
-int AFBClient::dispatch() {
-    return sd_event_run(loop, -1);
+int AFBClient::dispatch(uint64_t timeout) {
+    return sd_event_run(loop, timeout);
 }
 
 void AFBClient::deactivateSurface(const char *label)
 {
-    UNUSED(label);
+    printf("AFBClient::deactivateSurface(%s) -->\n", label);
+    fflush(stdout);
+    json_object *j = json_object_new_object();
+    json_object_object_add(j, "drawing_name", json_object_new_string(label));
+    call(AFBClient::wmAPI, "deactivate_surface", json_object_to_json_string(j));
+    json_object_put(j);
+    dispatch(-1);
+    printf("AFBClient::deactivateSurface(%s) <--\n", label);
+    fflush(stdout);
 }
 
 void AFBClient::endDraw(const char *label)
 {
-    UNUSED(label);
+    printf("AFBClient::endDraw(%s) -->\n", label);
+    fflush(stdout);
+    json_object *j = json_object_new_object();
+    json_object_object_add(j, "drawing_name", json_object_new_string(label));
+    call(AFBClient::wmAPI, "enddraw", json_object_to_json_string(j));
+    json_object_put(j);
+    dispatch(-1);
+    printf("AFBClient::endDraw(%s) <--\n", label);
+    fflush(stdout);
 }
 
 /* called when wsj1 receives a method invocation */
@@ -161,18 +208,6 @@ void AFBClient::onReply(void *closure, afb_wsj1_msg *msg)
     free(closure);
 }
 
-void AFBClient::onRequestSurfaceReply(void *closure, afb_wsj1_msg *msg)
-{
-    printf("onRequestSurfaceReply %s: %s\n%s\n", (char*)closure,
-           afb_wsj1_msg_is_reply_ok(msg) ? "OK" : "ERROR",
-           json_object_to_json_string_ext(afb_wsj1_msg_object_j(msg),
-                                          JSON_C_TO_STRING_PRETTY));
-    printf("\n\n===>RETURN STR: %s\n\n", afb_wsj1_msg_object_s(msg));
-    //    putenv("QT_IVI_SURFACE_ID=16778219");
-    fflush(stdout);
-    free(closure);
-}
-
 /* makes a call */
 void AFBClient::call(const char *api, const char *verb, const char *object)
 {