Re: [flac-dev] [PATCH] workaround for DJGPP missing wcswidth()

2017-01-13 Thread Erik de Castro Lopo
Ozkan Sezer wrote: > Attached patch works around for DJGPP missing wcswidth() > in flac/utils.c:strlen_console() Applied, thanks. Erik -- -- Erik de Castro Lopo http://www.mega-nerd.com/

Re: [flac-dev] unsigned int and FLAC__uint32 are used interchangeably

2017-01-13 Thread Erik de Castro Lopo
Ozkan Sezer wrote: > > Ozkan Sezer wrote: > > > >> unsigned int and FLAC__uint32 are used interchangeably, leading to > >> warnings with platforms (e.g. djgpp) where int32_t is long: > > > > Is `sizeof int == 4` though? > > Yes, obviously it is Well I've just pushed a patch that purges the code

Re: [flac-dev] unsigned int and FLAC__uint32 are used interchangeably

2017-01-13 Thread Ozkan Sezer
On 1/14/17, Erik de Castro Lopo wrote: > Ozkan Sezer wrote: > >> unsigned int and FLAC__uint32 are used interchangeably, leading to >> warnings with platforms (e.g. djgpp) where int32_t is long: > > Is `sizeof int == 4` though? Yes, obviously it is > I couldn't image FLAC

Re: [flac-dev] unsigned int and FLAC__uint32 are used interchangeably

2017-01-13 Thread Erik de Castro Lopo
Ozkan Sezer wrote: > unsigned int and FLAC__uint32 are used interchangeably, leading to > warnings with platforms (e.g. djgpp) where int32_t is long: Is `sizeof int == 4` though? I couldn't image FLAC working correctly if it wasn't. Erik --

[flac-dev] [PATCH 1/4] Do not override CFLAGS, as CFLAGS is a user flag.

2017-01-13 Thread David Seifert
* Furthermore, use NDEBUG globally to detect the presence of building with more debug output information. AX_CHECK_ENABLE_DEBUG is easier to use, and nowadays Gnome has also switched to it from its own custom solution. --- configure.ac | 19 +-- include/FLAC/assert.h

[flac-dev] [PATCH 3/4] Honor user's $(htmldir) and do not override GNU defaults for $(docdir)

2017-01-13 Thread David Seifert
* HTML files should be installed to $(htmldir), and $(docdir) should not be changed, as this is a user flag in the GNU conventions. --- doc/Makefile.am | 2 -- doc/html/Makefile.am| 10 -- doc/html/images/Makefile.am | 4 ++-- 3 files changed, 6 insertions(+), 10

[flac-dev] [PATCH 2/4] Make building/installing examples optional

2017-01-13 Thread David Seifert
--- Makefile.am | 6 +- configure.ac | 5 + 2 files changed, 10 insertions(+), 1 deletion(-) diff --git a/Makefile.am b/Makefile.am index a325e3a9..38aea04a 100644 --- a/Makefile.am +++ b/Makefile.am @@ -31,7 +31,11 @@ ACLOCAL_AMFLAGS = -I m4 -SUBDIRS = doc include m4 man src

[flac-dev] Upstreaming Gentoo patches

2017-01-13 Thread David Seifert
Dear FLAC devs, I would like to get some of our patches merged into master. Most of them deal with adhering to GNU conventions, respectively not overriding flags/variables that are up to the user to set. For instance, honoring $(htmldir) is important, as we have installation paths for the

[flac-dev] unsigned int and FLAC__uint32 are used interchangeably

2017-01-13 Thread Ozkan Sezer
unsigned int and FLAC__uint32 are used interchangeably, leading to warnings with platforms (e.g. djgpp) where int32_t is long: bitreader.c: In function `FLAC__bitreader_read_rice_signed': bitreader.c:721: warning: passing arg 2 of `FLAC__bitreader_read_unary_unsigned' from incompatible pointer

[flac-dev] [PATCH] workaround for DJGPP missing wcswidth()

2017-01-13 Thread Ozkan Sezer
Attached patch works around for DJGPP missing wcswidth() in flac/utils.c:strlen_console() -- O.S. 0001-flac-utils.c-strlen_console-workaround-for-DJGPP-mis.patch Description: Binary data ___ flac-dev mailing list flac-dev@xiph.org

[flac-dev] [PATCH] add missing string.h include to cpu.c (for memset())

2017-01-13 Thread Ozkan Sezer
Attached patch adds missing string.h include to cpu.c (for memset()) -- O.S. 0001-add-missing-string.h-include-to-cpu.c-for-memset.patch Description: Binary data ___ flac-dev mailing list flac-dev@xiph.org

[flac-dev] [PATCH] support nasm coff obj format for djgpp

2017-01-13 Thread Ozkan Sezer
Attached patch adds support nasm coff obj format for djgpp -- O.S. 0001-support-nasm-coff-obj-format-for-djgpp.patch Description: Binary data ___ flac-dev mailing list flac-dev@xiph.org http://lists.xiph.org/mailman/listinfo/flac-dev

Re: [flac-dev] [PATCH] configure.ac: relax linux OS detection

2017-01-13 Thread Erik de Castro Lopo
Peter Korsgaard wrote: > Not all linux hosts match the *-pc-linux-gnu wildcard, causing build > failures for older glibc versions where we need to link with -lrt for > clock_gettime - E.G.: Applied. Thanks. Erik -- -- Erik de

Re: [flac-dev] 1.3.2: FLAC__CPUINFO_IA32_CPUID_SSE3 undeclared

2017-01-13 Thread Erik de Castro Lopo
Christian Weisgerber wrote: > FWIW, here's a minimal fix to deal with this. Applied. Thanks. Erik -- -- Erik de Castro Lopo http://www.mega-nerd.com/ ___ flac-dev mailing list