[vlc-devel] [PATCH] skins2: Remove macOS code

Felix Paul Kühne fkuehne at videolan.org
Sun Dec 11 12:21:56 CET 2016


Hello,

> On 10 Dec 2016, at 18:24, Marvin Scholz <epirat07 at gmail.com> wrote:
> 
> Skins2 is not built on macOS since very long time, given that Qt is not
> used there. The code is very old with a lot of TODOs and additionally it
> requires the quite old Carbon framework which implies 32bit only as
> Carbon was never ported to 64bit. (Cocoa should be used instead)
> 
> Therefore this commit removes all macOS related skins2 code.

I agree with this patch. Considering that all our Objective-C code requires the modern runtime exclusive to 64bit user land (or 32bit iOS), we could not even easily compile VLC for 32bit Macs unless most Objective-C modules were disabled.

Carbon is a deprecated technology and even though a 64bit port was announced for macOS 10.5, it never made it in any public release.

If we ever decide to port the skins2 interface to macOS properly, it would need to be re-written using modern day technologies.

Best regards,

Felix


More information about the vlc-devel mailing list