<div dir="ltr"><div>Hi Thomas,</div><div><br></div><div>I was wrong when I said I don't need the core,</div><div>I still do (for pic creation when testing and benching vfilters, in this case the deinterlacers).</div><div><br></div><div>I see what you mean, yeah I think that should work. Will try it.</div><div>Thanks for the idea.<br></div><div><br></div><div>Victorien.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 14, 2020 at 9:47 PM Thomas Guillem <<a href="mailto:thomas@gllm.fr">thomas@gllm.fr</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"><br>
<br>
On Tue, Apr 14, 2020, at 18:45, Rémi Denis-Courmont wrote:<br>
> 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>
> <br>
> That's what I call taking reviewers for fools, and it really pisses me off.<br>
<br>
I found that asking advices to reviewers is very sane personally. I really don't see how Victorien patch set or comments can piss off someone else.<br>
<br>
<br>
Victorien, what about a common header file in modules that #undef / #define the requesting vlc_CPU. This header could be used by all your tests.<br>
<br>
> <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><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>