[vlc-devel] [PATCH v2] Add new vulkan/libplacebo-based vout
Niklas Haas
vlc at haasn.xyz
Wed Oct 10 03:46:32 CEST 2018
Version 2 of my vulkan/libplacebo vout patchset. Notable changes:
- Dropped the unrelated patches which are merged into master
- Split up vulkan/surface.c into platform-specific implementation files.
I'm not sure if using -DPLATFORN_NAME in the makefile like that is a
good idea or not?
Concerning some remaining feedback from the previous thread:
- I dropped the use of VLAs in favor of a fixed stack array of 128
pointers. This is not a significant size on any platform, and the
function is only called on init anyway.
- <config.h> is needed after all: it provides the N_ translation macro
- I kept the HAVE_LIBPLACEBO as it is. This is just a move patch, and I
don't think the issue (having to maybe rebuild vlc once more) is big
enough to warrant breaking consistency for its sake.
I accidentally sent a previous e-mail with this title while composing
the message, please, ignore.
More information about the vlc-devel
mailing list