#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)
Change History (11)
Changed 14 years ago by sharky@…
| Attachment: | patch-qaudio-format-converters.diff added |
|---|
Changed 14 years ago by sharky@…
| Attachment: | patch-qaudioinput-conversion.diff added |
|---|
Changed 14 years ago by sharky@…
| Attachment: | patch-qaudioinput-readyread.diff added |
|---|
Changed 14 years ago by sharky@…
| Attachment: | patch-qaudiooutput-period-size.diff added |
|---|
comment:1 Changed 14 years ago by mf2k (Frank Schima)
| Owner: | changed from macports-tickets@… to michaelld@… |
|---|
comment:2 Changed 14 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 14 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 14 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 14 years ago by michaelld (Michael Dickens)
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Sorry it took so long; been traveling a lot w/o reliable internet. I added these in r70280. Thanks!
comment:6 Changed 14 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 14 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 :)
Don't forget to assign to the maintainer.