search

jamoma / JamomaCore

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

Discuss existing TTDataInfo methods w Theo #217

Closed nwolek closed 8 years ago

nwolek commented 10 years ago

It looks like some parts of Modular duplicate functions that exist in the Foundation as TTDataInfo. It might be possible to replace with so that we are more DRY and then work on optimizing it in one place.

TTDataInfo stuff is found here: https://github.com/jamoma/JamomaCore/blob/master/Foundation/library/source/TTBase.cpp#L422 https://github.com/jamoma/JamomaCore/blob/master/Foundation/library/includes/TTBase.h#L253

@theod and @nwolek need to talk w @tap

raised during code review 5 March Bergen

nwolek commented 8 years ago

I reviewed this code again and no longer see the similarity. TTDataInfo code is doing something different than the Modular code, so the opportunity for optimization is not really there.