[vlc-devel] [PATCH v2 01/10] core: add atomic refcounter helper
Rémi Denis-Courmont
remi at remlab.net
Mon Jul 2 17:33:40 CEST 2018
I have to disagree and your bugs are not an excuse for somebody else to add bugs.
Fix your bugs.
Le 2 juillet 2018 18:30:31 GMT+03:00, Steve Lhomme <robux4 at ycbcr.xyz> a écrit :
>On 2018-07-02 17:25, Rémi Denis-Courmont wrote:
>> 1) This creates obvious overflow. Exactly the bug that the inspiring
>> kernel code is meant to fix...
>>
>> 2) So this won't work for e.g. picture_t.
>
>Then it won't.
>
>>
>> 3, 4) Inspiring kernel code does address those issues.
>>
>> And then, no assert in public headers.
>
>We already have 3 of these (one from me, two from Thomas). In VLC core
>headers it's fine as it's a controlled environment, even for external
>code. For libvlc it wouldn't be OK at all.
>
>>
>> Le 2 juillet 2018 18:18:27 GMT+03:00, "Hugo Beauzée-Luyssen"
>> <hugo at beauzee.fr> a écrit :
>>
>> On Mon, Jul 2, 2018, at 8:12 AM, Rémi Denis-Courmont wrote:
>>
>> Same problems as before. -- Envoyé de mon appareil Android
>> avec Courriel K-9 Mail. Veuillez excuser ma brièveté.
>>
>------------------------------------------------------------------------
>> vlc-devel mailing list To unsubscribe or modify your
>> subscription options:
>> https://mailman.videolan.org/listinfo/vlc-devel
>>
>>
>> - type sizes (uint, uintptr...),
>>
>>
>> The serie only affect code that was using uint, except for patch
>4 that was using uintptr, and I believe uint to be a better fit; and
>patch 10. I'm fine with dropping patch 10 for now.
>>
>> - order on decrement - acq_rel is not always needed,
>>
>>
>> I believe it is.
>>
>> - corner cases where the decrement is known to not reach
>zero,
>>
>>
>> Then don't use the helper in that case?
>>
>> - corner cases where the decrement cannot be permitted to
>> reach zero(e.g. vlc_object_t, IIRC).
>>
>>
>> Ditto.
>>
>>
>> --
>> Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez
>> excuser ma brièveté.
>>
>>
>> _______________________________________________
>> vlc-devel mailing list
>> To unsubscribe or modify your subscription options:
>> https://mailman.videolan.org/listinfo/vlc-devel
>
>_______________________________________________
>vlc-devel mailing list
>To unsubscribe or modify your subscription options:
>https://mailman.videolan.org/listinfo/vlc-devel
--
Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20180702/68d9d37b/attachment.html>
More information about the vlc-devel
mailing list