[Ffmpeg-cvslog] r6813 - trunk/libavcodec/eval.c
Sat Oct 28 17:07:08 CEST 2006
On Sat, Oct 28, 2006 at 11:39:56AM +0200, Oded Shimon wrote:
> On Sat, Oct 28, 2006 at 10:36:33AM +0100, M?ns Rullg?rd wrote:
> > Oded Shimon <ods15 at ods15.dyndns.org> writes:
> > > On Sat, Oct 28, 2006 at 08:46:13AM +0200, Oded Shimon wrote:
> > >> On Sat, Oct 28, 2006 at 12:16:26AM +0200, michael wrote:
> > >> [..]
> > >> > case e_div: return e->value * (d / d2);
> > >> > case e_add: return e->value * (d + d2);
> > >> > + case e_last:return d2;
> > >>
> > >> I'm fairly sure there should be an 'e->value *' here... try the expression
> > >> '5;-2'
> > >
> > > I was almost right, '-(1;2)'
> > What does that mean?
> ';' is a statement seperator, the first value is ignored, the second is
> used as the result of the expression. So, '(1;2)' returns simply '2'. The
> bug was that the '-' was ignored, and the overall expression returned '2'
> instead of '-2'. The reason for that is that the sign is stored always in
btw, what is your oppinion about getting rid of thet value* hack? and replace
it either by a negate(x) operation or pass the sign up and switch add to sub
(with neg() only if that isnt possible) ?
ive implemented both but they are a few lines of code more so iam not sure
if its a good idea or not
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is
More information about the ffmpeg-cvslog