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

Filip Roséen filip at atch.se
Thu Jul 12 23:24:57 CEST 2018


Yo,

On 2018-07-12 17:28, Hugo Beauzée-Luyssen wrote:

> I'm not sure what a proper name would be though... I can think of a few options:
> vlc_cxx.h 
> vlc_cxx_helpers.h (not sure I like the _helper part though)
> vlc_memory (probably doesn't mention the c++ part enough)

Could we use a more c++ish header-extension, like `.hpp` or `.hh`? In
 terms of name I personally like `utils` or `utilities` more than
 `helpers`.

Depending on how this turns out, it might also be wise to consider how
this will affect additional c++ headers in the future (if any), and if
it then makes sense to add them in a subdirectory instead of mixing
them all in one place. Just a thought.

/F
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20180712/a02548af/attachment.html>


More information about the vlc-devel mailing list