Displaying 20 results from an estimated 10000 matches similar to: "PATCH: typo in bitreader.c / bitwriter.c"
2015 Dec 16
2
about word size in bitreader/bitwriter
There are preprocessor definitions in bitreader.c and bitwriter.c:
/* Things should be fastest when this matches the machine word size */
/* WATCHOUT: if you change this you must also change the following #defines down to SWAP_BE_WORD_TO_HOST below to match */
/* WATCHOUT: there are a few places where the code will not work unless uint32_t is >= 32 bits wide */
#define
2015 Dec 20
2
about word size in bitreader/bitwriter
Erik de Castro Lopo wrote:
> The think in and ideal world we would a:
>
> * Make it work correctly FLAC__BYTES_PER_WORD == 8 and compare the performance
> with FLAC__BYTES_PER_WORD == 4.
> * If there is an statistically measurable performance, keep it, otherwise
> remove the FLAC__BYTES_PER_WORD == 8 code all together.
I'll try to do it, but I don't have a deep
2015 Dec 28
1
[PATCH 3] for bitwriter.c
1) Added assert
FLAC__ASSERT((bits == 32) || (val>>bits == 0))
It was inspired by https://sourceforge.net/p/flac/bugs/351/
Such situation should be detected at least.
2) Unnecessary assignments removed:
bw->accum = val;
bw->bits = 0;
(bw->bits is already 0 at this place).
3) Asserts like
FLAC__ASSERT(parameter < sizeof(unsigned)*8)
were changed to
2016 Dec 07
5
Do we need a pre-release?
Erik de Castro Lopo wrote:
> Unfortunately I've lost/forgotten my Xiph SVN password. While I get
> that sorted out for the real release, the pre-releases are here:
>
> http://mega-nerd.com/tmp/flac-1.3.2pre1-win.zip
> http://mega-nerd.com/tmp/flac-1.3.2pre1.tar.xz
Forgot to mention that I have tested this on x86_64/linux, armhf/linux
and powerpc/linux.
Erik
--
2015 Dec 28
1
[PATCH 4] for test_libFLAC/bitwriter.c
1) The definition of FLAC__BitWriter was updated with the
current text from /libFLAC/bitwriter.c (the text in comments was changed).
And the definition of TOTAL_BITS was made closer to the code from /libFLAC/bitwriter.c.
2) The values for 'words' and 'bits' values now calculated, not just some
magic constants.
3) Added FLAC__U64L() for 64-bit constants. It seems that it
2015 Dec 16
1
[PATCH] fix UB in bitwriter.c
The commit <http://git.xiph.org/?p=flac.git;a=commitdiff;h=e3c8095336249182b6a8871747df3cc99640103a>
fixed UB in bitwriter.c. But there are 2 unused functions -
FLAC__bitwriter_rice_bits() and FLAC__bitwriter_write_rice_signed()
that also have this UB. IMHO it makes sense to fix their code as well.
Maybe someone will use them in future.
-------------- next part --------------
A non-text
2014 Jun 27
4
Lets work towards a new version
Martijn van Beurden wrote:
> Like I reported just before the release of 1.3.0 (mail of Fri,
> 05 Apr 2013 08:25:10 +0200, to be specific), compiling on
> Raspbian (Debian Wheezy, GCC 4.6) returns quite some warnings of
> the type -Wcast-align.
>
> > CC lpc_intrin_sse2.lo
> > CC lpc_intrin_sse41.lo
> > CC md5.lo
> > md5.c: In function
2015 Dec 31
1
[PATCH] Support 64-bit brword/bwword
Here is the patch that allows to set FLAC__BYTES_PER_WORD to 8.
This is disabled by default though. To test FLAC__BYTES_PER_WORD==8:
open src/libFLAC/bitreader.c, src/libFLAC/bitwriter.c, src/test_libFLAC/bitwriter.c
and change '#if 1' to '#if 0'.
The value of FLAC__BYTES_PER_WORD must be the same in src/libFLAC/bitwriter.c
and src/test_libFLAC/bitwriter.c.
OTOH, their value in
2016 Jan 04
0
about word size in bitreader/bitwriter
On Sun, Dec 20, 2015 at 01:30:57PM +0300, lvqcl wrote:
> Erik de Castro Lopo wrote:
>
> > The think in and ideal world we would a:
> >
> > * Make it work correctly FLAC__BYTES_PER_WORD == 8 and compare the performance
> > with FLAC__BYTES_PER_WORD == 4.
> > * If there is an statistically measurable performance, keep it, otherwise
> > remove the
2017 Jan 19
4
[PATCH] Fix cppcheck warnings
---
src/libFLAC/bitreader.c | 4 ++--
src/libFLAC/bitwriter.c | 4 ++--
src/plugin_xmms/plugin.c | 2 +-
src/share/utf8/charset.c | 1 +
src/test_libFLAC++/encoders.cpp | 8 ++++----
src/test_libFLAC/decoders.c | 4 ++--
src/test_libFLAC/encoders.c | 8 ++++----
7 files changed, 16 insertions(+), 15 deletions(-)
diff --git a/src/libFLAC/bitreader.c
2015 Oct 07
2
Are pointers to FLAC__int32 and int interchangeable?
There are following functions in bitreader.c:
FLAC__bool FLAC__bitreader_read_unary_unsigned(FLAC__BitReader *br, unsigned *val);
FLAC__bool FLAC__bitreader_read_rice_signed(FLAC__BitReader *br, int *val, unsigned parameter);
FLAC__bool FLAC__bitreader_read_rice_signed_block(FLAC__BitReader *br, int vals[], unsigned nvals, unsigned parameter);
* function FLAC__bitreader_read_rice_signed():
2016 Jun 26
4
FLAC__SSE_OS change
lvqcl wrote:
> It doesn't know about uint32_t type, so the definition of cpu_xgetbv_x86() fails.
> It can be fixed by adding "#include share/compat.h" to cpu.c (or by using
> FLAC__uint32 from FLAC/ordinals.h).
Ok, added share/compat.h.
> When I fix this, the following problem occurs:
>
> error LNK2019: unresolved external symbol ___cpuidex referenced in
2013 Sep 08
7
PATCH: x86-64 support and SSE intrinscis code
It's not possible to use ia32/*.nasm code in 64-bit compiles.
There's still no 64-bit asm code in FLAC. I'm not familiar with asm too,
so I wrote SSE-accelerated code using intrinsics.
This code uses two new preprocessor macros:
FLAC__CPU_X86_64 (analogous to FLAC__CPU_IA32)
and FLAC__HAS_X86INTRIN (analogous to FLAC__HAS_NASM)
Patch for cpu.c/cpu.h adds CPU features (sse3, ssse3)
2013 Mar 09
9
flac 1.3.0pre2 pre-release
Hi all,
Second and hopefully final pre-release is here:
http://downloads.xiph.org/releases/flac/beta/
I have personally tested this code on:
x86-linux
x86_64-linux
powerpc-linux
armhf-linux
i386-freebsd9.1
i386-openbsd5.2
I also cross-compiled from Linux to 32 bit Windows and the compile ran to
completion (the test suite requires a bunch of hacking before it can
2016 Mar 09
2
Broken build on musl libc
Patch is here: https://github.com/openwrt/packages/commit/1263599f96f13f11d719ce336dfb6a639b32de98 . Probably needs to be modified for inclusion into mainline.
2013 Apr 07
2
flac 1.3.0pre3 pre-release
Dagobert Michelsen wrote:
> There compilation on Solaris 10 Sparc with Sun Studio 12 gives the following compile errors:
>
> > CC bitreader.lo
> > "bitreader.c", line 494: warning: implicit function declaration: MIN
> > CC bitwriter.lo
> > "bitwriter.c", line 273: reference to static identifier "bitwriter_grow_" in extern
2015 Oct 08
1
Are pointers to FLAC__int32 and int interchangeable?
Erik de Castro Lopo wrote:
> Well FLAC__int32 is just a 32 bit integer and on all the platforms/
> architecures/compilers that FLAC supports FLAC__int32 and int are
> the same.
>
> Personally I think the FLAC__xxxx stuff should go, to be replaced with
> C standard int32_t, uint32_t, int16_t etc, at least for internal code.
> I am slowly doing that when I touch code.
>
>
2015 Dec 28
6
How to check for 64-bit CPU?
In stream_encoder.c there's the following code:
#if defined FLAC__CPU_X86_64 /* and other 64-bit arch, too */
if(mean <= 0x80000000/512) { /* 512: more or less optimal for both 16- and 24-bit input */
#else
if(mean <= 0x80000000/8) { /* 32-bit arch: use 32-bit math if possible */
#endif
A) How to properly check for 64-bit architectures?
I can check for "defined
2016 Dec 07
2
Do we need a pre-release?
lvqcl.mail wrote:
> Microsoft Visual Studio 2015:
> format.c(48): error C2065: 'PACKAGE_VERSION': undeclared identifier
>
> I edited MSVC project files (I changed 'VERSION' preprocessor variable
> to 'PACKAGE_VERSION') and it resulted in successful build:
> "Rebuild All: 26 succeeded, 0 failed, 0 skipped"
Can I ask you to provide a patch
2014 Jul 09
3
[PATCH] PPC/Altivec removal
This set of patches removes PPC/Altivec code from FLAC.
I decided to split the patch into 5 parts to make it
more simple:
1) removes FLAC__lpc_restore_signal_asm_ppc_altivec_16*
from lpc.h and stream_decoder.c
2) removes PPC-specific code from cpu.c and cpu.h
3) removes PPC stuff from libFLAC/Makefile.lite and build/*.mk
4) removes as/gas/PPC-specific stuff from configure.ac
and