Re: [FFmpeg-devel] [PATCH 02/14] libavcodec: Implementation of AAC_fixed_decoder (LC-module) [2/5]

2014-10-10 Thread Nedeljko Babic
On Thu, Oct 09, 2014 at 12:02:26PM +, Nedeljko Babic wrote: softfloat uses if (a.mant + 0x4000 0) to normalize 0x4000U + 0x4000U is 0 for int32 and thus not part of the range though -1 would be, is that a problem ? we could use a.mant + 0x4000 = 0 in that case the

Re: [FFmpeg-devel] [PATCH 02/14] libavcodec: Implementation of AAC_fixed_decoder (LC-module) [2/5]

2014-10-09 Thread Nedeljko Babic
softfloat uses if (a.mant + 0x4000 0) to normalize 0x4000U + 0x4000U is 0 for int32 and thus not part of the range though -1 would be, is that a problem ? we could use a.mant + 0x4000 = 0 in that case the main difference i see to aac is that it shifts up if its too small while

Re: [FFmpeg-devel] [PATCH 02/14] libavcodec: Implementation of AAC_fixed_decoder (LC-module) [2/5]

2014-10-09 Thread Michael Niedermayer
On Thu, Oct 09, 2014 at 12:02:26PM +, Nedeljko Babic wrote: softfloat uses if (a.mant + 0x4000 0) to normalize 0x4000U + 0x4000U is 0 for int32 and thus not part of the range though -1 would be, is that a problem ? we could use a.mant + 0x4000 = 0 in that case the

Re: [FFmpeg-devel] [PATCH 02/14] libavcodec: Implementation of AAC_fixed_decoder (LC-module) [2/5]

2014-10-08 Thread Nedeljko Babic
Hi again, but I do agree that it is more maintainable to have one float emulation and I am willing to integrate our emulation in softfloat. However, there is a difference in some conventions used (for example is it more important to represent exactly 0.5 or 1, order of fields in struct

Re: [FFmpeg-devel] [PATCH 02/14] libavcodec: Implementation of AAC_fixed_decoder (LC-module) [2/5]

2014-10-07 Thread Michael Niedermayer
Hi On Mon, Oct 06, 2014 at 10:40:51AM +, Nedeljko Babic wrote: Hi and sorry for such a late response. It looks that I misplaced this mail... From: Djordje Pesut djordje.pe...@imgtec.com Add float emulation Signed-off-by: Nedeljko Babic nedeljko.ba...@imgtec.com ---

Re: [FFmpeg-devel] [PATCH 02/14] libavcodec: Implementation of AAC_fixed_decoder (LC-module) [2/5]

2014-09-16 Thread Michael Niedermayer
On Mon, Sep 01, 2014 at 07:55:40PM +0200, Nedeljko Babic wrote: From: Djordje Pesut djordje.pe...@imgtec.com Add float emulation Signed-off-by: Nedeljko Babic nedeljko.ba...@imgtec.com --- libavcodec/float_emu.h | 295 +

Re: [FFmpeg-devel] [PATCH 02/14] libavcodec: Implementation of AAC_fixed_decoder (LC-module) [2/5]

2014-09-02 Thread Nedeljko Babic
+/* Rounding to zero used for simplicity */ +static av_always_inline aac_float_t float_add(aac_float_t a, aac_float_t b) +{ +int diff; + +if (a.mant == 0) +return b; + +if (b.mant == 0) +return a; + +diff = a.expo - b.expo; + +if (diff 0) //

[FFmpeg-devel] [PATCH 02/14] libavcodec: Implementation of AAC_fixed_decoder (LC-module) [2/5]

2014-09-01 Thread Nedeljko Babic
From: Djordje Pesut djordje.pe...@imgtec.com Add float emulation Signed-off-by: Nedeljko Babic nedeljko.ba...@imgtec.com --- libavcodec/float_emu.h | 295 + libavcodec/float_emu_tab.c | 293 2 files

Re: [FFmpeg-devel] [PATCH 02/14] libavcodec: Implementation of AAC_fixed_decoder (LC-module) [2/5]

2014-09-01 Thread Reimar Döffinger
On Mon, Sep 01, 2014 at 07:55:40PM +0200, Nedeljko Babic wrote: +/* Rounding to zero used for simplicity */ +static av_always_inline aac_float_t float_add(aac_float_t a, aac_float_t b) +{ +int diff; + +if (a.mant == 0) +return b; + +if (b.mant == 0) +return a;

Re: [FFmpeg-devel] [PATCH 02/14] libavcodec: Implementation of AAC_fixed_decoder (LC-module) [2/5]

2014-08-08 Thread Nedeljko Babic
...@gmx.de] Poslato: 7. avgust 2014 22:12 Za: FFmpeg development discussions and patches Tema: Re: [FFmpeg-devel] [PATCH 02/14] libavcodec: Implementation of AAC_fixed_decoder (LC-module) [2/5] On Thu, Aug 07, 2014 at 09:51:36AM +, Nedeljko Babic wrote: +Q30(0.95), Q30(0.9922480620), Q30

Re: [FFmpeg-devel] [PATCH 02/14] libavcodec: Implementation of AAC_fixed_decoder (LC-module) [2/5]

2014-08-07 Thread Nedeljko Babic
Thanks on your review, On Fri, Aug 01, 2014 at 03:53:08PM +0200, Nedeljko Babic wrote: +#if !defined(_AAC_FLOAT_EMU_) +#define _AAC_FLOAT_EMU_ That is not out usual style 1) We use #ifndef 2) Identifiers starting with _ + upper case letter are reserved by POSIX and never should be used 3) The

Re: [FFmpeg-devel] [PATCH 02/14] libavcodec: Implementation of AAC_fixed_decoder (LC-module) [2/5]

2014-08-07 Thread Reimar Döffinger
On Thu, Aug 07, 2014 at 09:51:36AM +, Nedeljko Babic wrote: +Q30(0.95), Q30(0.9922480620), Q30(0.9846153846), Q30(0.9770992366), I'm a bit unsure btw. if this makes more sense than coding the converted numbers. It feels like it combines the disadvantages, since it neither

[FFmpeg-devel] [PATCH 02/14] libavcodec: Implementation of AAC_fixed_decoder (LC-module) [2/5]

2014-08-01 Thread Nedeljko Babic
From: Djordje Pesut djordje.pe...@imgtec.com Add float emulation Signed-off-by: Nedeljko Babic nedeljko.ba...@imgtec.com --- libavcodec/aac_float_emu.h | 698 + libavcodec/float_emu.h | 400 ++ libavcodec/float_emu_tab.c |

Re: [FFmpeg-devel] [PATCH 02/14] libavcodec: Implementation of AAC_fixed_decoder (LC-module) [2/5]

2014-08-01 Thread Reimar Döffinger
On Fri, Aug 01, 2014 at 03:53:08PM +0200, Nedeljko Babic wrote: +#if !defined(_AAC_FLOAT_EMU_) +#define _AAC_FLOAT_EMU_ That is not out usual style 1) We use #ifndef 2) Identifiers starting with _ + upper case letter are reserved by POSIX and never should be used 3) The naming convention is