QMMP tuning (diskuse)

[mkoubik]

A co to vecpat správcům příslušného SCM?

[Martin Tůma]

To (u změny ovládání) přijde na řadu až pokuď o to někdo bude mít zájem, tzn. uvidim tady komentáře, že to alespoň někdo vidí stejně jako já. A oprava řazení je takovej "hotfix" a chtělo by to dotyčnej kus kódu opravit víc "od podlahy", protože to co tam v současný době je nedává moc smysl...

[kavol]

To (u změny ovládání) přijde na řadu až pokuď o to někdo bude mít zájem, tzn. uvidim tady komentáře, že to alespoň někdo vidí stejně jako já.

hm, zkoušet to nebudu a co jsem pochopil z komentáře, je mi to buřt, nicméně pokud hodláš měnit nějaké chování, a to původní není chyba ale záměr, tak by to chtělo přidat možnost toto chování konfigurovat

A oprava řazení je takovej "hotfix" a chtělo by to dotyčnej kus kódu opravit víc "od podlahy", protože to co tam v současný době je nedává moc smysl...

mno, a zkoušel ses Ilji zeptat, jestli o té chybě ví? - možná by sis ušetřil práci s vymýšlením patche

[Martin Tůma]

hm, zkoušet to nebudu a co jsem pochopil z komentáře, je mi to buřt, nicméně pokud hodláš měnit nějaké chování, a to původní není chyba ale záměr, tak by to chtělo přidat možnost toto chování konfigurovat

To je právě těžký říct, Winamp, Audacious a vůbec každej "winamp-like" přehrávač kterej znám se chová tak jak to upravuje ten patch.

mno, a zkoušel ses Ilji zeptat, jestli o té chybě ví? - možná by sis ušetřil práci s vymýšlením patche

"Vymyslet" ten patch bylo poměrně triviální a výsledek se pro mě dostaví mnohem rychleji než kdyby se oprava měla objevit až v další verzi QMMP. Nicméně pokuď to v další verzi QMMP opravený nebude, bugreport autorovi pošlu.

[kavol]

... výsledek se pro mě dostaví mnohem rychleji než kdyby se oprava měla objevit až v další verzi QMMP.

ta oprava se samozřejmě objeví nejprve v svnku projektu, odkud není problém ji získat dřív než vyjde další verze ...

Nicméně pokuď to v další verzi QMMP opravený nebude, bugreport autorovi pošlu.

no pokud o tom neví, tak to asi opravený nebude :-p

[Přemek Jordák]

Příloha:

• PKGBUILD (1209 bytů)

Diky, zrovna se mi to hodi. Ale kdyz se otevira seznam skladeb qmmp file dialogem (ne defaultni Qt open file dialog) konci to neopravnenym pristupem do pameti (SIGSEGV). Jestli by nekdo chtel PKGBUILD pro Arch, tak je v priloze.

[Martin Tůma]

Hmm, dobrá připomínka. S QMMP file dialogem to nefunguje, neboť ten na rozdíl od "běžného" file dialogu neblokuje hlavní GUI, takže seřazení a spuštění se provede ihned na ještě prázdném seznamu. Upravil jsem ten patch alespoň tak, aby to při použití QMMP file dialogu nepadalo. (Chová se to "jako dřív")