<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 14, 2020 at 6:45 PM Rémi Denis-Courmont <<a href="mailto:remi@remlab.net">remi@remlab.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Le tiistaina 14. huhtikuuta 2020, 19.23.18 EEST Victorien Le Couviour a écrit <br>
:<br>
> On Tue, Apr 14, 2020 at 5:54 PM Rémi Denis-Courmont <<a href="mailto:remi@remlab.net" target="_blank">remi@remlab.net</a>> wrote:<br>
> > Le tiistaina 14. huhtikuuta 2020, 18.42.56 EEST Victorien Le Couviour a<br>
> > écrit<br>
> > <br>
> > > Hi,<br>
> > > <br>
> > > Doesn't that imply I'd have to write a version per feature I want to<br>
> > <br>
> > test?<br>
> > <br>
> > No but it does imply that you don't kludge the LibVLC run-time to suit<br>
> > your specific test. Like everybody else who's ever written a test case<br>
> > here.<br>
> <br>
> I don't see how this can work.<br>
<br>
So you can patch the Makefile to include select C file in your benchmark tool, <br>
but you cannot (think to) patch it to have different CFLAGS and a different <br>
vlc_CPU() implementation?<br></blockquote><div><br></div><div>I thought about it when I did that and rejected the idea,</div><div>but maybe it's because I was benching the deinterlacers differently and needed to link with the core for other purposes.</div><div>And so I couldn't have another implementation of vlc_CPU, as there were already 2 with a weak attribute on one of them, IIRC.</div><div><div>(that was many months ago now, the reflections behind those patches aren't so fresh in my mind anymore)</div><div></div></div><div><br></div><div>That's not the case anymore, and I don't think I need the core for anything else here anymore anyway, since I changed the way to bench that,</div><div>so it could be fine now, I need to actually look into it again.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
That's what I call taking reviewers for fools, and it really pisses me off.<br></blockquote><div><br></div><div>You should think about doing some meditation dude.</div><div>It's not normal the get "really pissed", as you say, over such things.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
-- <br>
Rémi Denis-Courmont<br>
Tapiola new town, Uusimaan Republic<br>
<br>
<br>
<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></div>