[vlc-devel] [PATCH] vlc_common: Add C++ memory management helpers

Filip Roséen filip at atch.se
Fri Jul 13 09:00:28 CEST 2018


Just noticed that *c++11* features were already introduced in
`<vlc_tick.h>`, so my vote might not count as the decision might have
been made already - but personally I strongly believe one should
protect such entities with a guard.

On 2018-07-13 07:28, Filip Roséen wrote:

> The above was meant to be read as *"Helpers are nice, but an
> unfortunate consequence of putting them in `vlc_common.h` is that the
> C++ compiler used to build a module then must support (and enable)
> *c++11*, even if the implementation of said module does not mandate
> it."*
> 
> > This could be circumvented by introducing `#if __cplusplus >= 201103`
> > so that we do not force an error diagnostic down someones throat who
> > are for whatever stuck with an older compiler. One could argue that we
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20180713/e10c4ba7/attachment.html>


More information about the vlc-devel mailing list