[Clam-devel] AudioFileMemoryLoader patch
tuuresairon at gmail.com
Tue Jun 24 12:44:05 PDT 2008
Files can be looped now. I've tested it quite thoroughly and I can
find no more errors.
On 23/06/2008, Pawel Bartkiewicz <tuuresairon at gmail.com> wrote:
> Some small fixes suggested by David:
> - added bounds to the position InControl
> - replaced redundant Configure with IsConfigured (Configure was being
> called twice for the inner MonoAudioFileReader)
> and a small fix in the header file.
> The loop option doesn't work yet.
> On 23/06/2008, Pawel Bartkiewicz <tuuresairon at gmail.com> wrote:
>> On 21/06/2008, David García Garzón <dgarcia at iua.upf.edu> wrote:
>>> in all platforms? Belive us, the delta is needed. There is arithmetic
>>> involved, when comparing you are substracting and comparing to 0. And it
>>> safer if you use .9/samplerate delta instead 1./samplerate. Just in case.
>> Ok, you're right, I haven't thought about that.
>>> Also include <cmath> instead <math.h> and std::fabs instead fabs. If
>>> no need for the include to be in the hxx move it to the cxx.
>> Sorry, I did that by mistake and didn't notice.
>>> I dont think that latest revisions solve that. The step value is the step
>>> the slider, not for the spinbox. A related todo is making the control
>>> have an step smaller than .01. Anyway when we address the feedback
>>> part, we could provide a better option.
>> Yes, it still doesn't work as expected in the newest revision.
>>> Just svn add it and provide it as part of the next patch.
>> Well... as you wish. I'm not sure how the binary .pos file will be
>> handled, though.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 1477 bytes
Desc: not available
More information about the clam-devel