Ticket #11922: 0004-Implement-BMediaRoster-GetFileFormatsFor.2.patch

headers/private/media/ServerInterface.h

@@ -134 +134 @@
     FILEINTERFACE_SET_REF,
     FILEINTERFACE_GET_REF,
     FILEINTERFACE_SNIFF_REF,
+    FILEINTERFACE_GET_FORMATS,
     FILEINTERFACE_MESSAGE_END,
 
     CONTROLLABLE_MESSAGE_START = 0x600,
@@ -995 +996 @@
     float                   capability;
 };
 
+struct fileinterface_get_formats_request : request_data {
+    int32                   num_formats;
+    area_id                 data_area;
+};
+
+struct fileinterface_get_formats_reply : reply_data {
+    int32                   filled_slots;
+};
 
 // #pragma mark - controllable commands
 

src/kits/media/FileInterface.cpp

@@ -97 +97 @@
 
             return B_OK;
         }
+        case FILEINTERFACE_GET_FORMATS:
+        {
+            const fileinterface_get_formats_request *request =
+                    (const fileinterface_get_formats_request*) data;
+            fileinterface_get_formats_reply reply;
+
+            media_file_format* formats;
+            area_id area = clone_area("client formats area",
+                (void**)&formats, B_ANY_ADDRESS, B_WRITE_AREA,
+                request->data_area);
+
+            if (area != B_OK) {
+                ERROR("FILEINTERFACE_GET_FORMATS: can't clone area");
+                break;
+            }
+
+            int32 cookie = 0;
+            while (GetNextFileFormat(&cookie, formats) == B_OK) {
+                if (cookie >= request->num_formats)
+                    break;
+                formats += sizeof(media_format);
+                cookie++;
+            }
+            reply.filled_slots = cookie;
+            request->SendReply(B_OK, &reply, sizeof(reply));
+
+            delete_area(area);
+            return B_OK;
+        }
         default:
             return B_ERROR;
     }

src/kits/media/MediaRoster.cpp

@@ -2815 +2815 @@
 BMediaRoster::GetFileFormatsFor(const media_node& fileInterface,
     media_file_format* _formats, int32* _numFormats)
 {
-    UNIMPLEMENTED();
-    return B_ERROR;
+    CALLED();
+
+    if (IS_INVALID_NODE(fileInterface)
+        || (fileInterface.kind & B_FILE_INTERFACE) == 0)
+        return B_MEDIA_BAD_NODE;
+
+    if (_numFormats == NULL || *_numFormats < 1)
+        return B_BAD_VALUE;
+
+    fileinterface_get_formats_request request;
+    fileinterface_get_formats_reply reply;
+    status_t rv;
+
+    media_file_format* formats;
+    size_t size = (size_t)(ceil(sizeof(media_file_format) * *_numFormats
+        / B_PAGE_SIZE) * B_PAGE_SIZE);
+
+    area_id area = create_area("formats area", (void**)&formats,
+        B_ANY_ADDRESS, size, B_NO_LOCK,
+        B_READ_AREA | B_WRITE_AREA);
+
+    if (area < B_OK)
+        return B_ERROR;
+
+    request.num_formats = *_numFormats;
+    request.data_area = area;
+
+    rv = QueryPort(fileInterface.port, FILEINTERFACE_GET_FORMATS, &request,
+        sizeof(request), &reply, sizeof(reply));
+    if (rv != B_OK) {
+        delete_area(area);
+        return rv;
+    }
+
+    memcpy(_formats, formats, sizeof(media_file_format)*reply.filled_slots);
+    *_numFormats = reply.filled_slots;
+
+    delete_area(area);
+    return B_OK;
 }