[FFmpeg-devel] [PATCH] libavutil: Make changes in softfloat needed for fixed point aac decoder.

Michael Niedermayer michaelni at gmx.at
Mon Apr 27 18:31:10 CEST 2015

On Mon, Apr 27, 2015 at 03:58:44PM +0200, Nedeljko Babic wrote:
> From: Djordje Pesut <djordje.pesut at imgtec.com>

> Functions for sqrt and sincos are added.

these could be in a seperate patch

> Div function is improved.

if its about adding av_recip_sf/av_div_sf_approx() then that commit
message is outdated and should be changed to match the code

> Some changes are made in order for code in softfloat to be usable in fixed aac
> decoder code.

which and why?

> This doesn't create any impact on current ffmpeg code since softfloat is
> currently not in use and this way we don't need to make much changes in
> implementation of aac fixed point decoder that uses this code.

> Softfloat tests are adjusted.

no, this just changed the header files, the tests where in the c

also the commit message should mention how exactly the Softfloat
exp/mantisa are changed, is this a bugfix, is there some offset
added, is some sign changed, ...
if you flip the sign or add a offset to some field that should be a
seperate patch ideally so it can easily be understood. Otherwise
if its mixed in with other changes the whole commit looks

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 1
"Used only once"    - "Some unspecified defect prevented a second use"
"In good condition" - "Can be repaird by experienced expert"
"As is" - "You wouldnt want it even if you were payed for it, if you knew ..."
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150427/fafb9e7d/attachment.asc>

More information about the ffmpeg-devel mailing list