[Clam-devel] [PATCH] TonalAnalysis configuration, take two

Roman Goj roman.goj at gmail.com
Thu Jul 12 07:13:44 PDT 2007


Sending the first in a new series (hopefully) of tiny patches adding
configuration to TonalAnalysis. First for _tunningEnabled (member of

In short this patch:
* Adds configuration for TonalAnalysis - _tunningEnabled is configurable
* Adds _tunningEnabled accessors in Simac::ChordExtractor

but, of course, a question:
I wrote the accessors enableTunning() and disableTunning() is this the
way to go or rather enableTunning(bool) ? I think I like the second
idea more, maybe more confusing if to disable tunning one would write
enableTunning(false). The second option would allow this code:

_implementation->enableTunning ( _config.GetTunningEnabled() )

instead of this:

is there a convention adopted in CLAM for doing this type of accessors?


PS, spent something like 5 hours yesterday battling with conflicting
sandboxes (couldn't even force the code to print Hello World ;) ),
what should've taken me 5 minutes took more then those 5 hours...
finally I have just one sandbox and so far so good...
-------------- next part --------------
A non-text attachment was scrubbed...
Name: TonalAnalysisConfig-step1-tunningEnabled.patch
Type: text/x-diff
Size: 2026 bytes
Desc: not available
URL: <http://lists.clam-project.org/pipermail/clam-devel-clam-project.org/attachments/20070712/3620e3fd/attachment-0003.patch>

More information about the clam-devel mailing list