Re: [mpg123-devel] Please test upcoming mpg123 1.32 (current snapshot or svn trunk)

[Thomas B. <to...@tr...>]

On 2023-08-21 22:13, Thomas Orgis wrote:
> TL;DR: Test mpg123 snapshot on your platform and report breakage. 
> Please.
> 
> 
> So, give
> 
> 	https://mpg123.org/snapshot
> 
> a spin. The full list of NEWS scheduled for the upcoming release 
> contains
> some more points:
> 

Hello, I've gave it a spin (x86_64, Linux) and everything looks good so 
far. Only a small set of functions are involved so it might not be 
representative.

Something observed independently and most likely not related to the new 
snapshot is that when compile flag
--enable-gapless=no
is set, then mpg123_seek seems to not work. This also happens with for 
example 1.31.3. The effect is that a mpg123_seek(h, 10, SEEK_SET) will 
return 0.
When --enable-gapless=no is not set, it will return 10, nothing else 
changed.
I'm not sure how gapless is related to seek, I couldn't pin it down.

Greetings
Thomas


PS, here the quick & dirty testcode:
#include <stdio.h>
#include <mpg123.h>
#include <stdint.h>
#include <inttypes.h>

static mpg123_handle* mh;

//=====================================================
int main(int argc, char **argv)
{
	long rate;
	int channels, format, err;
	int64_t offset=10;
	int64_t pos=0;

	mpg123_init();
	mh = mpg123_new(NULL, NULL);
	mpg123_format_none (mh);
	format=mpg123_format(mh
		, 44100
		, MPG123_STEREO
		, MPG123_ENC_FLOAT_32
	);
	err = mpg123_open(mh, argv[1]);
	if(err == MPG123_OK)
	{
		mpg123_getformat(mh, &rate, &channels, &format);

		pos=mpg123_seek(mh,offset,SEEK_SET);
		fprintf(stderr, "seek end return %" PRId64 "\n", pos);

		mpg123_close(mh);
	}
	else{fprintf(stderr, "Failed to open file: %s\n", 
mpg123_strerror(mh));}

	mpg123_exit();
}

./configure
vs
./configure --enable-gapless=no