Bug 183520 - KCompactDisc doesn't honor asynchronous mode
Summary: KCompactDisc doesn't honor asynchronous mode
Status: ASSIGNED
Alias: None
Product: kdemultimedia
Classification: Miscellaneous
Component: general (show other bugs)
Version: unspecified
Platform: openSUSE Unspecified
: NOR normal
Target Milestone: ---
Assignee: Multimedia Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-02-07 06:41 UTC by Robby Stephenson
Modified: 2021-11-04 15:00 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Robby Stephenson 2009-02-07 06:41:53 UTC
Version:            (using KDE 4.2.0)
Installed from:    SuSE RPMs

The Phonon backend for the KCompactDisc class doesn't seem to honor the KCompactDisc::InformationMode set in the KCompactDisc constructor. It's always Synchronous, even if the Asynchronous mode is set.

If I grep for infomode in the libkcompactdisc directory:

kde4@homebase:~/src/kdemultimedia/libkcompactdisc> grep -i infomode *.cpp
kcompactdisc.cpp:KCompactDisc::KCompactDisc(InformationMode infoMode) :
kcompactdisc.cpp:    d->m_infoMode = infoMode;
kcompactdisc_p.cpp:    m_infoMode(KCompactDisc::Synchronous),
kcompactdisc_p.cpp:     pNew->m_infoMode = m_infoMode;
wmlib_interface.cpp:            if (m_infoMode == KCompactDisc::Asynchronous) {

Seems like the phonon_interface.cpp class should check m_infoMode like the wmlib_interface does.
Comment 1 Harald Sitter 2011-08-14 22:33:54 UTC
Well, if I understand the code correctly I very much doubt we would be able to support this with Phonon because it leaves the decision up to the backend. Which would be rather difficult to change for the 4.x series.

Needs some looking into and discussion though.
Comment 2 Justin Zobel 2021-03-09 23:42:42 UTC
Thank you for the bug report.

As this report hasn't seen any changes in 5 years or more, we ask if you can please confirm that the issue still persists.

If this bug is no longer persisting or relevant please change the status to resolved.
Comment 3 Robby Stephenson 2021-11-04 15:00:37 UTC
I believe this is still present (and undocumented) in the Phonon backend, but KCompactDisc doesn't show up with much usage on lxr.kde.org