jamoma / JamomaCore

Jamoma Frameworks for Audio and Control Structure
Other
36 stars 14 forks source link

Refactor out 6 deprecation warnings in TTMixer #225

Closed nwolek closed 10 years ago

nwolek commented 10 years ago

/Volumes/Storage/xcode_projects/Jamoma/Core/DSP/library/source/TTMixer.cpp /Volumes/Storage/xcode_projects/Jamoma/Core/DSP/library/source/TTMixer.cpp:35:2: 'TTObjectBaseInstantiate' is deprecated /Volumes/Storage/xcode_projects/Jamoma/Core/DSP/library/source/TTMixer.cpp:36:2: 'TTObjectBaseInstantiate' is deprecated /Volumes/Storage/xcode_projects/Jamoma/Core/DSP/library/source/TTMixer.cpp:37:2: 'TTObjectBaseInstantiate' is deprecated /Volumes/Storage/xcode_projects/Jamoma/Core/DSP/library/source/TTMixer.cpp:63:2: 'TTObjectBaseRelease' is deprecated /Volumes/Storage/xcode_projects/Jamoma/Core/DSP/library/source/TTMixer.cpp:64:2: 'TTObjectBaseRelease' is deprecated /Volumes/Storage/xcode_projects/Jamoma/Core/DSP/library/source/TTMixer.cpp:65:2: 'TTObjectBaseRelease' is deprecated

nwolek commented 10 years ago

still there as of commit 999924f

https://github.com/jamoma/JamomaCore/blob/merge/api_and_api-cleanup/DSP/library/source/TTMixer.cpp

nwolek commented 10 years ago

Tried to employ the new TTMatrixBase constructor, but I now see it is protected so that is obviously wrong. @tap: can you look at this one and see how you would recommend employing new Matrix syntax?

nwolek commented 10 years ago

OK, now I see some models for the revised syntax here: https://github.com/jamoma/JamomaCore/commit/8d553affdc75155325d7920d9bf87a683fd9dd31 https://github.com/jamoma/JamomaCore/commit/9ee5851c7ca37e1fcb32f7c0fed72fac008c201e https://github.com/jamoma/JamomaCore/commit/67ca0b799354678391cbeba20074ad9801a3bdae

tap commented 10 years ago

I changed one of the 3 TTMatrix instances in my above commit. If you need more then please let me know.

nwolek commented 10 years ago

Yes, thanks! I updated another and now will complete the third.