vorbis-tools-flac.patch

[Jakub Bogusz]

On Wed, Jan 03, 2007 at 10:58:28PM +0100, Szymon Siwek wrote:
> On Wed, Jan 03, 2007 at 10:20:37PM +0100, Jakub Bogusz wrote:
> > On Wed, Jan 03, 2007 at 10:33:03AM +0100, Szymon Siwek wrote:
> > > Witam!
> > > 
> > > Mam zainstalowany zestaw vorbis-tools & flac-1.1.3.
> > > ogg123 hgw.flac kończy się segfaultem.
> > > Czy to jest wyjątek, czy reguła?
> > 
> > Może chodzi o to:
> > https://trac.xiph.org/changeset/12220
> > czyli błędne założenie o liczbie kanałów (2), powodujące wywalanie się
> > na plikach mono. W SVN poprawione.
> > 
> To było moje pierwsze podejrzenie, bo testowałem akurat na pliku mono. 
> Ale nie o to chodziło.
> Tobie twoja wersja działa bez problemu?

Nie wiem, nie mam żadnego pliku flac. Pisałem na podstawie zmian API,
może po prostu coś przeoczyłem.

> > > Jeśli reguła, to proponuję zastąpić aktualny vorbis-tools-flac.patch
> > > załączonym (+poprawka speca /patch3/s/p1/p2/). Oczywiście gdy ktoś poza mną
> > > przetestuje.
> > 
> > Zmiany niepotrzebnie skomplikowane przez warunkowe pozostawienie
> > easyflaca dla starego flaca... ale skoro tak jest w SVN-ie, to OK.
> Cóż, łatka może skomplikowana, poprostu zapisałem odpowiedni changeset. 
> 
> > Faktyczne różnice (na plus) w stosunku do mojej łaty są dwie:
> > - ww. poprawka na liczbę kanałów != 2
> - prawdopodobnie można by zmniejszyć BR: flac-devel
> 
> > - implementacja eof_callback
> > 
> Nie jestem przekonany, czy ta różnica powodowała segfaulta - u mnie
> wysypywało się od razu, nie przy końcu nagrania.

Funkcja może być wcześniej wywoływana - ona ma zwracać, czy jest
koniec pliku/strumienia.
Ale wg opisu API jest opcjonalna (do _init można przekazać NULL).


-- 
Jakub Bogusz    http://qboosh.pl/