Re: [patch] Replace internal mpglib with libmpg123 usage

Thomas Orgis <[email protected]> Wed, 13 Sep 2017 16:11:38 +0200
Newsgroups gmane.comp.audio.mp3.lame
Message-ID <20170913161138.198c0b6e@pocke>
Am Tue, 12 Sep 2017 19:22:41 +0200
schrieb Thomas Orgis <[email protected]>: 

> Am Tue, 12 Sep 2017 18:02:14 +0200
> schrieb "Robert Hegemann" <[email protected]>: 
> 
> > Using VC2008 and short of time, I haven't even managed to get libmpg123 compiled.  
> 
> Hm, VC2008 might be troublesome.

OK, what about trying to use a fresh libmpg123 DLL from

	https://mpg123.org/download/win32/r4351/
	https://mpg123.org/download/win64/r4351/

? They are freshly built using MinGW and it is what people under
Windows generally use, if they do not feel the urge to build everything
under MSVC++ for the heck of it.

I am not sure if the use of fileno() and mpg123_open_fd() works out on
Windows. I do not have a clear picture on the versions of C library
there and the validity of integer file descriptors. Maybe we need to
switch to mpg123_open_handle() (callback input) … or we could
spoon-feed the data like mpglib, but that is less efficient.

Can someone perhaps confirm on *NIX first that the patch does not only
work for me? Then we can handle portability. There's mpg123 on OS/2,
you know? ;-)


Alrighty then,

Thomas
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Lame-dev mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/lame-dev