Changeset 83621 in vbox for trunk/src/VBox/HostServices/SharedClipboard

Timestamp:

Apr 8, 2020 3:02:16 PM (5 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

137049

Message:

SharedClipboard/darwin: Logging. bugref:9620

Location:

trunk/src/VBox/HostServices/SharedClipboard

Files:

• VBoxSharedClipboardSvc.cpp (modified) (3 diffs)
• darwin-pasteboard.cpp (modified) (5 diffs)

trunk/src/VBox/HostServices/SharedClipboard/VBoxSharedClipboardSvc.cpp

@@ -1209 +1209 @@
     while (fFormats)
     {
-        SHCLFORMAT fFormat = VBOX_SHCL_FMT_NONE;
-
+        /* Pick the next format to get from the mask: */
         /** @todo Make format reporting precedence configurable? */
+        SHCLFORMAT fFormat;
         if (fFormats & VBOX_SHCL_FMT_UNICODETEXT)
             fFormat = VBOX_SHCL_FMT_UNICODETEXT;
@@ -1219 +1219 @@
             fFormat = VBOX_SHCL_FMT_HTML;
         else
-            AssertStmt(fFormats == VBOX_SHCL_FMT_NONE, fFormat = VBOX_SHCL_FMT_NONE);
-
-        if (fFormat == VBOX_SHCL_FMT_NONE)
-            break;
-
-        /* Remove format from format list. */
+            AssertMsgFailedBreak(("%#x\n", fFormats));
+
+        /* Remove it from the mask. */
         fFormats &= ~fFormat;
 
@@ -1343 +1340 @@
 int ShClSvcHostReportFormats(PSHCLCLIENT pClient, SHCLFORMATS fFormats)
 {
-    LogFlowFuncEnter();
-    AssertPtrReturn(pClient,  VERR_INVALID_POINTER);
+    LogFlowFunc(("fFormats=%#x\n", fFormats));
+    AssertPtrReturn(pClient, VERR_INVALID_POINTER);
 
 #ifdef VBOX_WITH_SHARED_CLIPBOARD_TRANSFERS

trunk/src/VBox/HostServices/SharedClipboard/darwin-pasteboard.cpp

@@ -85 +85 @@
 int queryNewPasteboardFormats(PasteboardRef pPasteboard, uint32_t *pfFormats, bool *pfChanged)
 {
-    Log(("queryNewPasteboardFormats\n"));
-
     OSStatus err = noErr;
+    *pfFormats = 0;
     *pfChanged = true;
 
@@ -97 +96 @@
     {
         *pfChanged = false;
+        Log2(("queryNewPasteboardFormats: no change\n"));
         return VINF_SUCCESS;
     }
@@ -104 +104 @@
     err = PasteboardGetItemCount(pPasteboard, &itemCount);
     if (itemCount < 1)
+    {
+        Log(("queryNewPasteboardFormats: changed: No items on the pasteboard\n"));
         return VINF_SUCCESS;
+    }
 
     /* The id of the first element in the pasteboard */
@@ -121 +124 @@
             {
                 CFStringRef flavorType;
-                flavorType = static_cast <CFStringRef>(CFArrayGetValueAtIndex(flavorTypeArray,
-                                                                               flavorIndex));
+                flavorType = static_cast <CFStringRef>(CFArrayGetValueAtIndex(flavorTypeArray, flavorIndex));
                 /* Currently only unicode supported */
-                if (UTTypeConformsTo(flavorType, kUTTypeUTF8PlainText) ||
-                    UTTypeConformsTo(flavorType, kUTTypeUTF16PlainText))
-                {
-                    Log(("Unicode flavor detected.\n"));
+                if (   UTTypeConformsTo(flavorType, kUTTypeUTF8PlainText)
+                    || UTTypeConformsTo(flavorType, kUTTypeUTF16PlainText))
+                {
+                    Log(("queryNewPasteboardFormats: Unicode flavor detected.\n"));
                     *pfFormats |= VBOX_SHCL_FMT_UNICODETEXT;
                 }
                 else if (UTTypeConformsTo(flavorType, kUTTypeBMP))
                 {
-                    Log(("BMP flavor detected.\n"));
+                    Log(("queryNewPasteboardFormats: BMP flavor detected.\n"));
                     *pfFormats |= VBOX_SHCL_FMT_BITMAP;
                 }
@@ -140 +142 @@
     }
 
-    Log(("queryNewPasteboardFormats: rc = %02X\n", rc));
+    Log(("queryNewPasteboardFormats: changed: *pfFormats=%#x\n", *pfFormats));
     return rc;
 }