RFR: 8232589: Remove CoreAudio Utility Classes
Kevin Rushforth
kcr at openjdk.java.net
Mon Jan 6 23:50:46 UTC 2020
On Mon, 6 Jan 2020 23:36:50 GMT, Alexander Matveev <almatvee at openjdk.org> wrote:
>> modules/javafx.media/src/main/native/jfxmedia/platform/osx/avf/AVFAudioEqualizer.cpp line 422:
>>
>>> 421: mEQBufferA = (double*)calloc(mEQBufferSize, sizeof(double));
>>> 422: mEQBufferB = (double*)calloc(mEQBufferSize, sizeof(double));
>>> 423: }
>>
>> This is allocating 8 times (i.e., `sizeof(double)`) as much memory as before. I presume this is intentional?
>
> No, it is allocating same amount of memory as before, unless I am missing something.
> We used to have CAAutoFree<double> mEQBufferA and
> CAAutoFree.alloc(size_t numItems) method allocates (numItems * sizeof(T)) bytes. See deleted CAAutoDisposer.h for CAAutoFree class implementation.
Yes, you're right. I missed that the previous `alloc` call was a method call on a CAAutoFree object, which does exactly what you said above. The new code is equivalent.
-------------
PR: https://git.openjdk.java.net/jfx/pull/69
More information about the openjfx-dev
mailing list