Cristian Rodríguez
2012-Mar-29 17:32 UTC
[flac-dev] [GIT PULL] Assorted bugfixes and improvements (from openSUSE)
The following changes since commit b78d8e4db10e57b8d82bb82e4e3662d5dedd7255:
FLAC__bitmath_ilog2,FLAC__bitmath_ilog2_wide,COUNT_ZERO_MSBS: add gcc
specific optimizations (2012-03-28 15:43:48 -0300)
are available in the git repository at:
git://github.com/crrodriguez/flac.git master
for you to fetch changes up to 3a060556772c5d6a6464afddfda7c3ad2f93a306:
Remove winamp2 plugin. (2012-03-29 13:51:24 -0300)
----------------------------------------------------------------
Cristian Rodr?guez (4):
Use a single source-code wide definition of min() and max()
Support symbol visibility features present in GCC > 4
Remove xmms plugin. (project abandoned in 2007)
Remove winamp2 plugin(native flac support is in winamp 5.3 6yr/old
configure.ac | 89 +--
include/FLAC++/export.h | 7 +-
include/FLAC/export.h | 26 +-
m4/visibility.m4 | 77 ++
src/Makefile.am | 6 -
src/flac/Makefile.am | 2 +-
src/flac/encode.c | 29 +-
src/flac/foreign_metadata.c | 7 +-
src/libFLAC++/Makefile.am | 3 +-
src/libFLAC++/metadata.cpp | 4 +
src/libFLAC++/stream_decoder.cpp | 4 +
src/libFLAC++/stream_encoder.cpp | 4 +
src/libFLAC/Makefile.am | 6 +-
src/libFLAC/bitreader.c | 13 +-
src/libFLAC/bitwriter.c | 11 +-
src/libFLAC/fixed.c | 16 +-
src/libFLAC/lpc.c | 16 +-
src/libFLAC/metadata_iterators.c | 15 +-
src/libFLAC/ogg_decoder_aspect.c | 7 +-
src/libFLAC/stream_decoder.c | 7 +-
src/libFLAC/stream_encoder.c | 48 +-
src/libFLAC/stream_encoder_framing.c | 5 -
src/metaflac/Makefile.am | 2 +-
src/plugin_common/Makefile.am | 48 --
src/plugin_common/Makefile.lite | 36 -
src/plugin_common/README | 2 -
src/plugin_common/all.h | 26 -
src/plugin_common/charset.c | 157 ----
src/plugin_common/charset.h | 39 -
src/plugin_common/defs.h | 24 -
src/plugin_common/dither.c | 262 -------
src/plugin_common/dither.h | 29 -
src/plugin_common/plugin_common_static.dsp | 128 ----
src/plugin_common/plugin_common_static.vcproj | 209 ------
src/plugin_common/replaygain.c | 64 --
src/plugin_common/replaygain.h | 32 -
src/plugin_common/tags.c | 358 ---------
src/plugin_common/tags.h | 74 --
src/plugin_winamp2/Makefile.am | 33 -
src/plugin_winamp2/configure.c | 428 -----------
src/plugin_winamp2/configure.h | 49 --
src/plugin_winamp2/in_flac.c | 443 ------------
src/plugin_winamp2/in_flac.dsp | 154 ----
src/plugin_winamp2/in_flac.vcproj | 253 -------
src/plugin_winamp2/include/Makefile.am | 18 -
src/plugin_winamp2/include/winamp2/Makefile.am | 20 -
src/plugin_winamp2/include/winamp2/in2.h | 107 ---
src/plugin_winamp2/include/winamp2/out.h | 55 --
src/plugin_winamp2/infobox.c | 459 ------------
src/plugin_winamp2/infobox.h | 28 -
src/plugin_winamp2/playback.c | 307 --------
src/plugin_winamp2/playback.h | 92 ---
src/plugin_winamp2/resource.h | 47 --
src/plugin_winamp2/resource.rc | 236 ------
src/plugin_winamp2/tagz.cpp | 921
------------------------
src/plugin_winamp2/tagz.h | 32 -
src/plugin_xmms/Makefile.am | 70 --
src/plugin_xmms/Makefile.lite | 43 --
src/plugin_xmms/charset.c | 198 -----
src/plugin_xmms/charset.h | 56 --
src/plugin_xmms/configure.c | 823
---------------------
src/plugin_xmms/configure.h | 77 --
src/plugin_xmms/fileinfo.c | 492 -------------
src/plugin_xmms/http.c | 899
-----------------------
src/plugin_xmms/http.h | 26 -
src/plugin_xmms/locale_hack.h | 55 --
src/plugin_xmms/plugin.c | 684 ------------------
src/plugin_xmms/plugin.h | 24 -
src/plugin_xmms/tag.c | 154 ----
src/plugin_xmms/tag.h | 24 -
src/share/grabbag/replaygain.c | 32 +-
src/test_libFLAC++/encoders.cpp | 4 +
src/test_libFLAC++/metadata_object.cpp | 4 +
src/test_libs_common/Makefile.am | 4 +-
src/test_libs_common/file_utils_flac.c | 7 +-
75 files changed, 229 insertions(+), 9021 deletions(-)
create mode 100644 m4/visibility.m4
delete mode 100644 src/plugin_common/Makefile.am
delete mode 100644 src/plugin_common/Makefile.lite
delete mode 100644 src/plugin_common/README
delete mode 100644 src/plugin_common/all.h
delete mode 100644 src/plugin_common/charset.c
delete mode 100644 src/plugin_common/charset.h
delete mode 100644 src/plugin_common/defs.h
delete mode 100644 src/plugin_common/dither.c
delete mode 100644 src/plugin_common/dither.h
delete mode 100644 src/plugin_common/plugin_common_static.dsp
delete mode 100644 src/plugin_common/plugin_common_static.vcproj
delete mode 100644 src/plugin_common/replaygain.c
delete mode 100644 src/plugin_common/replaygain.h
delete mode 100644 src/plugin_common/tags.c
delete mode 100644 src/plugin_common/tags.h
delete mode 100644 src/plugin_winamp2/Makefile.am
delete mode 100644 src/plugin_winamp2/configure.c
delete mode 100644 src/plugin_winamp2/configure.h
delete mode 100644 src/plugin_winamp2/in_flac.c
delete mode 100644 src/plugin_winamp2/in_flac.dsp
delete mode 100644 src/plugin_winamp2/in_flac.vcproj
delete mode 100644 src/plugin_winamp2/include/Makefile.am
delete mode 100644 src/plugin_winamp2/include/winamp2/Makefile.am
delete mode 100644 src/plugin_winamp2/include/winamp2/in2.h
delete mode 100644 src/plugin_winamp2/include/winamp2/out.h
delete mode 100644 src/plugin_winamp2/infobox.c
delete mode 100644 src/plugin_winamp2/infobox.h
delete mode 100644 src/plugin_winamp2/playback.c
delete mode 100644 src/plugin_winamp2/playback.h
delete mode 100644 src/plugin_winamp2/resource.h
delete mode 100644 src/plugin_winamp2/resource.rc
delete mode 100644 src/plugin_winamp2/tagz.cpp
delete mode 100644 src/plugin_winamp2/tagz.h
delete mode 100644 src/plugin_xmms/Makefile.am
delete mode 100644 src/plugin_xmms/Makefile.lite
delete mode 100644 src/plugin_xmms/charset.c
delete mode 100644 src/plugin_xmms/charset.h
delete mode 100644 src/plugin_xmms/configure.c
delete mode 100644 src/plugin_xmms/configure.h
delete mode 100644 src/plugin_xmms/fileinfo.c
delete mode 100644 src/plugin_xmms/http.c
delete mode 100644 src/plugin_xmms/http.h
delete mode 100644 src/plugin_xmms/locale_hack.h
delete mode 100644 src/plugin_xmms/plugin.c
delete mode 100644 src/plugin_xmms/plugin.h
delete mode 100644 src/plugin_xmms/tag.c
delete mode 100644 src/plugin_xmms/tag.h
Cristian Rodríguez
2012-Mar-29 17:57 UTC
[flac-dev] [GIT PULL] Assorted bugfixes and improvements (from openSUSE)
El 29/03/12 14:32, Cristian Rodr?guez escribi?:> The following changes since commit > b78d8e4db10e57b8d82bb82e4e3662d5dedd7255: > > FLAC__bitmath_ilog2,FLAC__bitmath_ilog2_wide,COUNT_ZERO_MSBS: add gcc > specific optimizations (2012-03-28 15:43:48 -0300) > > are available in the git repository at: > > git://github.com/crrodriguez/flac.git master > > for you to fetch changes up to 3a060556772c5d6a6464afddfda7c3ad2f93a306: > > Remove winamp2 plugin. (2012-03-29 13:51:24 -0300) > > ---------------------------------------------------------------- > Cristian Rodr?guez (4): > Use a single source-code wide definition of min() and max() > Support symbol visibility features present in GCC > 4 > Remove xmms plugin. (project abandoned in 2007) > Remove winamp2 plugin(native flac support is in winamp 5.3 6yr/old >Please pull from commit b78d8e4db10e57b8d82bb82e4e3662d5dedd7255 including that particular commit , I made a mistake in the "since" field.
Erik de Castro Lopo
2012-Mar-30 10:14 UTC
[flac-dev] [GIT PULL] Assorted bugfixes and improvements (from openSUSE)
Cristian Rodr?guez wrote:> The following changes since commit b78d8e4db10e57b8d82bb82e4e3662d5dedd7255: > > FLAC__bitmath_ilog2,FLAC__bitmath_ilog2_wide,COUNT_ZERO_MSBS: add gcc > specific optimizations (2012-03-28 15:43:48 -0300) > > are available in the git repository at: > > git://github.com/crrodriguez/flac.git master > > for you to fetch changes up to 3a060556772c5d6a6464afddfda7c3ad2f93a306:Hi Cristian, Thanks for your interest in improving FLAC. I will address your patches one-by-one. * b78d8e4db1.. FLAC__bitmath_ilog2,FLAC__bitmath_ilog2_wide,COUNT_ZERO_MSBS: add gcc? +1. You sent me this patch in a private email and I was in the process of commiting a slightly modified version. * 7077ba8be2.. Use a single source-code wide definition of min() and max() I like the idea of this commit but don't understand why it modifies configure.ac. I also don't think the FLAC_SAFE_MIN/MAX defintions belong in the file include/FLAC/export.h which is a public header file. Instead, this definition belongs in one of the private/internal header files. * 8f05d0114c.. Support symbol visibility features present in GCC > 4 Again I like the idea of this patch, but I had in mind a solution to this problem which will work for a number of compilers other than just GCC. * f2b0d3f6c9.. Remove xmms plugin. I polled the list earlier this year and on the basis of one of the Xiph developers still using this code and being willing to maintain it, we decided to keep it. * 3a06055677.. Remove winamp2 plugin. This one I fully endorse, but is unlikely to apply without the patches above being applied first. I'll apply a version of this patch. Cheers, Erik -- ---------------------------------------------------------------------- Erik de Castro Lopo http://www.mega-nerd.com/
Cristian Rodríguez
2012-Mar-30 13:04 UTC
[flac-dev] [GIT PULL] Assorted bugfixes and improvements (from openSUSE)
El 30/03/12 07:14, Erik de Castro Lopo escribi?:> * 7077ba8be2.. Use a single source-code wide definition of min() and max() > > I like the idea of this commit but don't understand why it modifies > configure.ac. I also don't think the FLAC_SAFE_MIN/MAX defintions > belong in the file include/FLAC/export.h which is a public header file. > Instead, this definition belongs in one of the private/internal header > files.Yes, I didnt find the right header ;)> > * 8f05d0114c.. Support symbol visibility features present in GCC> 4 > > Again I like the idea of this patch, but I had in mind a solution to > this problem which will work for a number of compilers other than just > GCC.The current version will work with clang, icc and gcc those define __GNUC__ and accept the -fvisibility-hidden flag
Cristian Rodríguez
2012-Mar-30 16:01 UTC
[flac-dev] [GIT PULL] Assorted bugfixes and improvements (from openSUSE)
El 30/03/12 07:14, Erik de Castro Lopo escribi?:> Cristian Rodr?guez wrote: > >> The following changes since commit b78d8e4db10e57b8d82bb82e4e3662d5dedd7255: >> >> FLAC__bitmath_ilog2,FLAC__bitmath_ilog2_wide,COUNT_ZERO_MSBS: add gcc >> specific optimizations (2012-03-28 15:43:48 -0300) >> >> are available in the git repository at: >> >> git://github.com/crrodriguez/flac.git master >> >> for you to fetch changes up to 3a060556772c5d6a6464afddfda7c3ad2f93a306: > > Hi Cristian, > > Thanks for your interest in improving FLAC. I will address your patches > one-by-one. > > * b78d8e4db1.. FLAC__bitmath_ilog2,FLAC__bitmath_ilog2_wide,COUNT_ZERO_MSBS: add gcc? > > +1. You sent me this patch in a private email and I was in the process > of commiting a slightly modified version.there is a tiny problem there. the preprocessor check for __builtin_clz #if defined(__GNUC__) && ((__GNUC__ == 3 && __GNUC_MINOR__ >= 4) || __GNUC__ >= 4) So it compiles on very old systems...
Seemingly Similar Threads
- Errors compiling flac in Visual Studio Express 2005
- Re: beta 10 candidate checked in
- [GIT PULL] Assorted bugfixes and improvements (from openSUSE)
- [PATCH] Fix building with MSYS and MinGW(-w64); Improve Makefile.lite build system
- [PATCH] Makefile.lite: Fix building with MSYS and MinGW(-w64), Improvements