Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#25831 closed defect (fixed)

qt4-mac: QAudioInput and QAudioOutput are broken

Reported by: sharky@… Owned by: michaelld (Michael Dickens)
Priority: Normal Milestone:
Component: ports Version: 1.9.1
Keywords: Cc:
Port: qt4-mac

Description (last modified by sharky@…)

Up to and including version 4.6.3 of Qt, the QAudioInput and QAudioOutput classes of QtMultimedia are barely usable on OS X (Snow Leopard for sure). The issues have been fixed upstream and the fixes should be present in Qt 4.6.4, but it might be worth backporting the fixes. The attached patches do just that.

Attachments (4)

patch-qaudio-format-converters.diff (1.8 KB) - added by sharky@… 10 years ago.
patch-qaudioinput-conversion.diff (3.4 KB) - added by sharky@… 10 years ago.
patch-qaudioinput-readyread.diff (1.2 KB) - added by sharky@… 10 years ago.
patch-qaudiooutput-period-size.diff (1.8 KB) - added by sharky@… 10 years ago.

Download all attachments as: .zip

Change History (11)

Changed 10 years ago by sharky@…

Changed 10 years ago by sharky@…

Changed 10 years ago by sharky@…

Changed 10 years ago by sharky@…

comment:1 Changed 10 years ago by mf2k (Frank Schima)

Owner: changed from macports-tickets@… to michaelld@…

Don't forget to assign to the maintainer.

comment:2 Changed 10 years ago by michaelld (Michael Dickens)

Thanks for the diffs. I'll try to check them out in the next few days. Do you happen to know if they're compatible with 10.4 / 10.5, or just meant for 10.6? IIRC, Apple made significant positive changes to the CoreAudio framework between 10.5 and 10.6 -- but, for many developers we just had a rename variable types & functions to the new API (not change the variables & arguments themselves).

comment:3 Changed 10 years ago by sharky@…

Description: modified (diff)

They should be compatible with all the OS X versions that Qt supports (10.4, 10.5, 10.6) since these patches are from the official Qt repository. I have personally only tested 10.6 but should be able to give you feedback on 10.5 if you want.

comment:4 Changed 10 years ago by michaelld (Michael Dickens)

They at least compile on 10.5, so that's good enough for me. Thanks for the head's up. I'll check in changes shortly.

comment:5 Changed 10 years ago by michaelld (Michael Dickens)

Resolution: fixed
Status: newclosed

Sorry it took so long; been traveling a lot w/o reliable internet. I added these in r70280. Thanks!

comment:6 Changed 10 years ago by nerdling (Jeremy Lavergne)

I'd suggest bumping the revision since Qt would have previously installed without issue. No one with Qt already installed would be able to use these patches until a rebuild.

comment:7 Changed 10 years ago by michaelld (Michael Dickens)

I've got some other changes to the qt4-mac* Portfiles to make, and I'll rev-bump at the end. I wanted to keep the checkins separate. I also forgot to add the patchfiles :)

Note: See TracTickets for help on using tickets.