Re: [FFmpeg-devel] [PATCH] lavfi/af_aemphasis: remove unnecessary complex number usage

2015-12-23 Thread Ganesh Ajjanagadde
On Wed, Dec 23, 2015 at 8:04 AM, Ganesh Ajjanagadde wrote: > On Tue, Dec 22, 2015 at 11:32 PM, Paul B Mahol wrote: >> On 12/22/15, Ganesh Ajjanagadde wrote: >>> On Tue, Dec 22, 2015 at 2:10 PM, Paul B Mahol wrote: On 12/22/15, Ganesh Ajjanagadde wrote: > On Tue, Dec 22, 2015 at 12:46

Re: [FFmpeg-devel] [PATCH] lavfi/af_aemphasis: remove unnecessary complex number usage

2015-12-23 Thread Ganesh Ajjanagadde
On Tue, Dec 22, 2015 at 11:32 PM, Paul B Mahol wrote: > On 12/22/15, Ganesh Ajjanagadde wrote: >> On Tue, Dec 22, 2015 at 2:10 PM, Paul B Mahol wrote: >>> On 12/22/15, Ganesh Ajjanagadde wrote: On Tue, Dec 22, 2015 at 12:46 AM, Paul B Mahol wrote: > On 12/22/15, Ganesh Ajjanagadde wr

Re: [FFmpeg-devel] [PATCH] lavfi/af_aemphasis: remove unnecessary complex number usage

2015-12-22 Thread Paul B Mahol
On 12/22/15, Ganesh Ajjanagadde wrote: > On Tue, Dec 22, 2015 at 2:10 PM, Paul B Mahol wrote: >> On 12/22/15, Ganesh Ajjanagadde wrote: >>> On Tue, Dec 22, 2015 at 12:46 AM, Paul B Mahol wrote: On 12/22/15, Ganesh Ajjanagadde wrote: > complex is not available on all platforms. Further

Re: [FFmpeg-devel] [PATCH] lavfi/af_aemphasis: remove unnecessary complex number usage

2015-12-22 Thread Ganesh Ajjanagadde
On Tue, Dec 22, 2015 at 2:10 PM, Paul B Mahol wrote: > On 12/22/15, Ganesh Ajjanagadde wrote: >> On Tue, Dec 22, 2015 at 12:46 AM, Paul B Mahol wrote: >>> On 12/22/15, Ganesh Ajjanagadde wrote: complex is not available on all platforms. Furthermore, it is trivial to rewrite complex nu

Re: [FFmpeg-devel] [PATCH] lavfi/af_aemphasis: remove unnecessary complex number usage

2015-12-22 Thread Paul B Mahol
On 12/22/15, Ganesh Ajjanagadde wrote: > On Tue, Dec 22, 2015 at 12:46 AM, Paul B Mahol wrote: >> On 12/22/15, Ganesh Ajjanagadde wrote: >>> complex is not available on all platforms. Furthermore, it is trivial to >>> rewrite complex number expressions to real arithmetic, and in fact >>> sometim

Re: [FFmpeg-devel] [PATCH] lavfi/af_aemphasis: remove unnecessary complex number usage

2015-12-22 Thread Ganesh Ajjanagadde
On Tue, Dec 22, 2015 at 12:46 AM, Paul B Mahol wrote: > On 12/22/15, Ganesh Ajjanagadde wrote: >> complex is not available on all platforms. Furthermore, it is trivial to >> rewrite complex number expressions to real arithmetic, and in fact >> sometimes advantageous for performance reasons: by wr

Re: [FFmpeg-devel] [PATCH] lavfi/af_aemphasis: remove unnecessary complex number usage

2015-12-22 Thread Paul B Mahol
On 12/22/15, Ganesh Ajjanagadde wrote: > complex is not available on all platforms. Furthermore, it is trivial to > rewrite complex number expressions to real arithmetic, and in fact > sometimes advantageous for performance reasons: by wrapping as a complex, > one forces a particular Cartesian rep

[FFmpeg-devel] [PATCH] lavfi/af_aemphasis: remove unnecessary complex number usage

2015-12-21 Thread Ganesh Ajjanagadde
complex is not available on all platforms. Furthermore, it is trivial to rewrite complex number expressions to real arithmetic, and in fact sometimes advantageous for performance reasons: by wrapping as a complex, one forces a particular Cartesian representation that is not necessarily optimal for