These are chat archives for BicycleStudio/Ewe

31st
Oct 2015
Sergey Vaytsel
@Telhar
Oct 31 2015 15:26
Валь, ревью по Sound готово, можно смотреть?
И да, там был баг с WindowFacade, он поправлен? На него задача заведена?
Valentin Zaborcev
@Swipes13
Oct 31 2015 15:34
  1. как таковая задача использовать dsound, да , посмотри
предлагаю ввести задачи на использование ogg
а с этой задачей покончить по этому ревью
  1. баг сейчас заведу )
Sergey Vaytsel
@Telhar
Oct 31 2015 15:35
давай сделаем задачу на ogg
сейчас буду думать на счёт задач, что сегодня обсуждали
Valentin Zaborcev
@Swipes13
Oct 31 2015 15:38
ogg и bugFacade завёл!
Sergey Vaytsel
@Telhar
Oct 31 2015 15:38
:smile: :+1:
Sergey Vaytsel
@Telhar
Oct 31 2015 16:40
Завёл 4 задачи в backlog, одну в ready, прокомментировал задачу #10, написал, что мы сегодня в конце концов решили. Можешь посмотреть, пожалуйста, вдруг я о чём то забыл.
Valentin Zaborcev
@Swipes13
Oct 31 2015 16:50
сейчас займусь)
Sergey Vaytsel
@Telhar
Oct 31 2015 18:16
ТОП-1 комментарий в ревью - "логгирование"
Sergey Vaytsel
@Telhar
Oct 31 2015 18:30
после того как сделаем MVP я займусь конфигами и тестами. без тестов нельзя, а то накопится большой техдолг
Valentin Zaborcev
@Swipes13
Oct 31 2015 19:20
сейчас всё исправлю :) ток домой зашёл
Valentin Zaborcev
@Swipes13
Oct 31 2015 20:31
вроде всё привёл в форму! получилось 24 коммита, а у нас 76, и если мерджить , будет ровно 100 коммитов!)) прошу обратить внимание: 1.я создал простенькую структурку, чтобы быстро инициализировать
WavFormat. вынес отдельно, туда же вынес WavHeader.
  1. в классах DShow::Audio и DSound::Audio сделал макрос с error для того, чтобы он не фаталил систему. ну нет файла и ладно, ошибка записалась, вывелась, если мы в дебаге, но играть, к примеру, дальше можно. прямо в макрос поместил функцию shutdown. интересно, что ты думаешь по такому ходу. по сути, если где-то ошибка произошла, то тога то, что проинициализировано будет проверкой удалено. и "сверху", там где мы экземпляр создавали, нам нужно только деструктор вызвать, чтобы убить логер
  1. полностью убрал "тестинг" системы, пока событий нет и объектов нет, к которым данные привязать, нет смысла что-то проигрывать)
гиттер почему-то постоянно делает мой список с единицы, хотя я пишу там 2. 3.
ну да ладно)
Valentin Zaborcev
@Swipes13
Oct 31 2015 23:30
есть тут один нюанс, если я хочу переключиться на работу с другой задачкой, например с звука на баг с WindowFacade, я должен был "откатываться" на мастера, делать новую ветку и, наконец, работать. но я продолжал свою, в силу того, что код, который был реализован для звука многое затронул в других файлах. Надеюсь, в будущем, когда система будет хорошо поставлена в основных её частях, таких проблем не будет. а пока придётся делать так. смотри пуллРеквест со звуком, думаю, там немного у тебя вопросов будет. я быстренько переделаю и идём на логирование графики. и после - на баг, в общем, последовательно)