Page MenuHomePhabricator

[ios] Implement getVideoInfo() function on iOS
ClosedPublic

Authored by angelika on Mar 20 2025, 4:33 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 17, 9:21 PM
Unknown Object (File)
Sun, Apr 13, 11:11 PM
Unknown Object (File)
Sun, Apr 13, 12:20 PM
Unknown Object (File)
Sat, Apr 12, 3:19 AM
Unknown Object (File)
Fri, Apr 11, 2:57 AM
Unknown Object (File)
Thu, Apr 10, 10:15 AM
Unknown Object (File)
Tue, Apr 8, 12:35 PM
Unknown Object (File)
Sat, Apr 5, 11:25 PM
Subscribers

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

angelika held this revision as a draft.
native/expo-modules/comm-expo-package/ios/MediaModule.swift
4 ↗(On Diff #47481)
71 ↗(On Diff #47481)

I don't know what format is in the original ffmpeg implementation. It's not documented anywhere and when I've tested it the ffmpeg implementation didn't even return it. Because it's later compared to "mp4" to check if format is correct I'm just returning the file extension. We can run more complicated checks but I think it's good enough for our usecase.

native/expo-modules/comm-expo-package/ios/MediaModule.swift
63 ↗(On Diff #47481)

According to the docs link you posted above, isn't this way of accessing format descriptions deprecated?

This revision is now accepted and ready to land.Sun, Mar 23, 11:47 PM
native/expo-modules/comm-expo-package/ios/MediaModule.swift
63 ↗(On Diff #47481)

It is deprecated but the new way is available for ios 15+ and we're supporting ios 13+

This revision was landed with ongoing or failed builds.Thu, Mar 27, 12:21 PM
This revision was automatically updated to reflect the committed changes.