On Wed, Apr 29, 2015 at 12:13:53PM +0000, Nedeljko Babic wrote:
> Ok, I'll split this patch some more and since this takes already too long I
> will eliminate any change that is not absolutely necessary for implementation
> of fixed point aac decoder (I am thinking here on new recip and div 
> functions).
> 
> Since there will be two or three new patches from this patch I'll resubmit 
> them
> as a new patch set (as oppose resending them as replay to this thread).
> 
> Regarding comments about changes to exponent, it is change that is needed for
> this soft float to be usable in implementation of fixed point aac decoder with
> the new functions that we are adding.
> 
> I could also change the value of ONE_BITS and adjust the code accordingly, but
> this comes down to the same.
> 
> The alternative would be to change the code that is already written (and
> probably to recalculate some of the tables) but that is much more work than 
> just
> changing exponent here.
> Since softfloat is not used anywhere currently, I don't see how this can be
> justified.
> 

> Maybe the comment in commit message should specify that exponent is changed 
> for
> softfloat to be usable in fixed aac decoder, or something similar?

yes and that patch should preferably change nothing else than the
exponent

Thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Democracy is the form of government in which you can choose your dictator

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to