Opened 8 years ago

Closed 8 years ago

Last modified 7 years ago

#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)

comment:1 by dimag, 8 years ago

Resolution: fixed
Status: newclosed

r522
Не могу, эксперимент пригодился для закрытия тикета 504, в ревизии r522, я изменил аудиовывод для системных звуков. Теперь он идет не через класс QSoundEffect, а через класс QAudioOutput, поэтому я вернул классы QAudioOutput и QAudioDeviceInfo.

comment:2 by san, 8 years ago

Resolution set to fixed

Не могу, эксперимент пригодился

В таком случае следовало закрыть тикет с резолюцией invalid, раз он неверен.

in reply to:  1 comment:3 by alx, 8 years ago

Resolution: fixed
Status: closedreopened

Replying to dimag:

Не могу,

Не можете, но закрываете тикет с резолюцией fixed? Явное противоречие... Определитесь, пожалуйста.

я изменил аудиовывод для системных звуков. Теперь он идет не через класс QSoundEffect,

Тогда поясните, пожалуйста, роль члена alarmSound класса MainAppFrameWindow3 в r522. Он как раз имеет класс QSoundEffect.

comment:4 by dimag, 8 years ago

Сейчас он не используется в r522, в r521 и более ранних он использовался для воспроизведение звука.

in reply to:  4 comment:5 by alx, 8 years ago

Replying to dimag:

Сейчас он не используется в r522,

Тогда почему Вы не удалили его из класса MainAppFrameWindow3? Стоит, наверное, удалить?

Last edited 8 years ago by alx (previous) (diff)

comment:6 by dimag, 8 years ago

Оставлю, вдруг будут какие-то проблемы с QAudioOutput, тогда будет к чему вернуться.
А так 1 экземпляр данного класса займёт совсем немного памяти и не будет больше использоваться.

comment:7 by dimag, 8 years ago

Resolution: invalid
Status: reopenedclosed

comment:8 by san, 7 years ago

Milestone: Текущее2 очередь

Milestone renamed

Note: See TracTickets for help on using tickets.