#517 closed улучшение (invalid)
Убрать следы "эксперимента" из исходного кода
| Reported by: | alx | Owned by: | dimag |
|---|---|---|---|
| Priority: | major | Milestone: | 2 очередь |
| Component: | ПО MC04-Dispatcher. Пульт диспетчера/техника | Keywords: | garbage |
| Cc: | san |
Description
По словам Дмитрия в комментарии к #514, он экспериментировал с классами QAudioDeviceInfo и QAudioOutput. Судя по всему, следы этих экспериментов по ошибке не были полностью убраны из исходного текста, когда делался коммит changeset:519. Эти следы остались в коде до сих пор (r522).
Необходимо их убрать.
Change History (8)
follow-up: 3 comment:1 by , 8 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:2 by , 8 years ago
Resolution set to fixed
Не могу, эксперимент пригодился
В таком случае следовало закрыть тикет с резолюцией invalid, раз он неверен.
comment:3 by , 8 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
Replying to dimag:
Не могу,
Не можете, но закрываете тикет с резолюцией fixed? Явное противоречие... Определитесь, пожалуйста.
я изменил аудиовывод для системных звуков. Теперь он идет не через класс QSoundEffect,
Тогда поясните, пожалуйста, роль члена alarmSound класса MainAppFrameWindow3 в r522. Он как раз имеет класс QSoundEffect.
follow-up: 5 comment:4 by , 8 years ago
comment:5 by , 8 years ago
Replying to dimag:
Сейчас он не используется в r522,
Тогда почему Вы не удалили его из класса MainAppFrameWindow3?
comment:6 by , 8 years ago
Оставлю, вдруг будут какие-то проблемы с QAudioOutput, тогда будет к чему вернуться.
А так 1 экземпляр данного класса займёт совсем немного памяти и не будет больше использоваться.
comment:7 by , 8 years ago
| Resolution: | → invalid |
|---|---|
| Status: | reopened → closed |
r522
Не могу, эксперимент пригодился для закрытия тикета 504, в ревизии r522, я изменил аудиовывод для системных звуков. Теперь он идет не через класс QSoundEffect, а через класс QAudioOutput, поэтому я вернул классы QAudioOutput и QAudioDeviceInfo.