
<html><head></head><body>Hi,<br><br>Using _Generic while preserving the promotion rules over all three parameter types is going to be horrible. And it will make the macro non-constant, which might break some other uses.<br><br>And ultimately, why the heck should VLC developers have to remember to use VLC_CLIP for min-max? There are enough VLC idiosyncrasies that one needs to learn and remember, not to add gratuitous ones.<br><br><div class="gmail_quote">Le 7 avril 2021 11:07:08 GMT+03:00, Alexandre Janniaux <ajanni@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">Hi,<br><br>On Wed, Apr 07, 2021 at 07:29:21AM +0200, Steve Lhomme wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">On 2021-04-06 16:25, Rémi Denis-Courmont wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;">Le tiistaina 6. huhtikuuta 2021, 17.07.01 EEST Steve Lhomme a écrit :<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #8ae234; padding-left: 1ex;">Why ?<br></blockquote>Because it's a poor excuse of a functional macro that is not expansion-safe,<br>and whose behaviour is utterly trivial.<br></blockquote>I disagree. This is the perfect use case for it. And replacing a one liner<br>by 3 lines is never good in my book.<br></blockquote><br>I too prefer VLC_CLIP, but it's non-obvious that it has been<br>misused here, whereas it is with condition so the change from<br>Marvin makes sense.<br><br>Maybe we can switch to using a _Generic/template<> inline<br>vlc_clip instead too?<br><br>Regards,<hr>Alexandre Janniaux<br>Videolabs<hr>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>