[Clam-devel] TokenDelay patch

Xavier Amatriain xavier at create.ucsb.edu
Fri May 4 14:00:21 PDT 2007


Uhm... I just realized that my optimized implementation is potentially
dangerous. The issue is that, until you reach the first delay point
(that is if delay is set to 200 ms until you reach that point in time)
you are getting default constructed objects out of the delay. If you are
working with spectrums, for instance, you get zero-sized spectrums. This
is potentially dangerous but I can't see an immediate fix.


On Sat, 2007-04-21 at 14:04 -0700, Xavier Amatriain wrote:
> Pau Arumi wrote:
> > david, i see a std::vector not an array.
> >
> > svn diff -r10038:10039
> > -       std::deque<T> mTokenQueue;
> > +       std::vector<T> mTokenQueue;
> >
> > but also
> > +#include "Array.hxx"
> >
> > i guess this include was added by mistake, right xavi?
> > (and the commit log was also kindof misleading)
> 
> Yes, sorry about that. I tried specializing he template for DataArrays
> but found it was not really needed. This include is just a remainder of that
> and should be removed. Will do.
> 
> _______________________________________________
> Clam-devel mailing list
> Clam-devel at llistes.projectes.lafarga.org
> https://llistes.projectes.lafarga.org/cgi-bin/mailman/listinfo/clam-devel
-- 
/*********************************
 *       Xavier Amatriain        *
 *  Associate Director - MATi    *
 *  Research Director - CREATE   *
 *    UCSB, Santa Barbara CA     *
 *      1-(805)- 893 83 52       *
 ********************************/





More information about the clam-devel mailing list