[Clam-devel] [PATCH] TonalAnalysis configuration, take two
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...
Size: 2026 bytes
Desc: not available
More information about the clam-devel