[Asterisk-code-review] bridge softmix binaural: Enable FFTW3 in Solaris 11. (asterisk[master])

Dennis Guse asteriskteam at digium.com
Thu Jun 28 01:30:33 CDT 2018


Dennis Guse has posted comments on this change. ( https://gerrit.asterisk.org/9289 )

Change subject: bridge_softmix_binaural: Enable FFTW3 in Solaris 11.
......................................................................


Patch Set 1:

Looks good so far.
Just for my understanding, what is the reason to move from double *fftw_alloc_real(size_t n) to fftw_malloc(sizeof(double) * n)?

http://www.fftw.org/fftw3_doc/Memory-Allocation.html


-- 
To view, visit https://gerrit.asterisk.org/9289
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ice5640e08385a64a0a6555deaccd91e86bca154f
Gerrit-Change-Number: 9289
Gerrit-PatchSet: 1
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Dennis Guse <dennis.guse at alumni.tu-berlin.de>
Gerrit-Reviewer: Frank Haase <fra.haase at googlemail.com>
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-Comment-Date: Thu, 28 Jun 2018 06:30:33 +0000
Gerrit-HasComments: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180628/236eb1f5/attachment.html>


More information about the asterisk-code-review mailing list