<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div>Hi,</div><div><br></div><div>Do you really need to use block_t for simple memory allocation ? For example in `send_rtcp_feedback`, it seems to me that malloc will do fine.</div><div><br></div><div>If you do need to guard against cancellation, you can use `vlc_cleanup_push(free, ptr);` instead.<br></div><div><br></div><div>I also find it a little bit confusing to understand what is a cancellation point and what is not, maybe you could add a comment close to where you use `rist_WriteTo`, or rename it to `rist_WriteTo_i11e` ?</div><div><br></div><div>Otherwise, +1 for str*cpy.<br></div><div><br></div><div>Best regards,</div><div>-- <br></div><div>Alexandre Janniaux<br></div></div></div></div></div><br><div class="gmail_quote"><div dir="ltr">Le lun. 22 oct. 2018 à 09:39, Thomas Guillem <<a href="mailto:thomas@gllm.fr">thomas@gllm.fr</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
On Mon, Oct 22, 2018, at 09:23, Thomas Guillem wrote:<br>
> <br>
> On Mon, Oct 22, 2018, at 02:42, Jean-Baptiste Kempf wrote:<br>
> > <br>
> > On Sun, 21 Oct 2018, at 16:57, Sergio M. Ammirata wrote:<br>
> > > All issues are fixed, here is in the revised input module patch.<br>
> > <br>
> > So:<br>
> > - missing NEWS, po/POTFILES.in, modules/MODULE_LIST entries<br>
> > - you must surround the poll.h include with HAVE_POLL<br>
> <br>
> > - use str*cpy instead of strcpy<br>
> <br>
> Where strcpy is used, dst and src have the same length. I don't know the <br>
> convention, but for me, strcpy is good in that case.<br>
<br>
Forget it, I though the src param was static const, but it's not. So yes, strcpy is bad here.<br>
<br>
> <br>
> > <br>
> > Also, you have many options in this module. Are they all necessary?<br>
> > <br>
> > best,<br>
> > <br>
> > <br>
> > -- <br>
> > Jean-Baptiste Kempf -  President<br>
> > +33 672 704 734<br>
> > _______________________________________________<br>
> > vlc-devel mailing list<br>
> > To unsubscribe or modify your subscription options:<br>
> > <a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
> _______________________________________________<br>
> vlc-devel mailing list<br>
> To unsubscribe or modify your subscription options:<br>
> <a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a></blockquote></div>