
<html><head></head><body>There was a protection and you removed it. By definition, we can obviously not have more uintptr_max in-memory refs.<br>

<br>

Now how does that thinking work with uint_max??<br><br><div class="gmail_quote">Le 2 juillet 2018 18:36:34 GMT+03:00, Romain Vimont <rom1v@videolabs.io> a écrit :<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<pre class="k9mail">On Mon, Jul 02, 2018 at 06:25:57PM +0300, Rémi Denis-Courmont wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;"> 1) This creates obvious overflow.<br></blockquote><br>There was no protection against refcount overflow before. This helper<br>does not add a protection against refcount overflow.<br><br>It seems to me that usages of atomic_uintptr_t for refcounters were just<br>an inconsistency, not justified by atomic_uint being not large enough.<br>And if it were the case, atomic_uintptr_t is not appropriate.<br><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;"> <br> 2) So this won't work for e.g. picture_t.<br></blockquote><br>Patch 3 use the refcounter for picture_priv_t. What's the problem?<br><hr><br>vlc-devel mailing list<br>To unsubscribe or modify your subscription options:<br><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a></pre></blockquote></div><br>

-- <br>

Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.</body></html>