Неиспользуемые функции MC04-DSL-VIP.

[mixyil1.1]

Институт "Агат" проводит экспертизу MC04-DSL-VIP в профильной организации. В ходе проверки возник вопрос о том, что в исходном коде встречается код для тех возможностей,
которые не используются в MC04-DSL-VIP, например:

управление конференцией:
source:comcerto.h#L249
source:comcerto.h#L250
source:comcerto.h#L251
source:comcerto.h#L252
канальное окончание EM01
source:em01.h
и т.д.

Это рассматривается как недокументированные возможности шлюза, которых не должно быть.

В связи с чем институт "Агат" просит рассмотреть возможность ревизии исходного кода на предмет удаления кода относящегося к тем возможностям платы VE-01, которые не используются в MC04-DSL-VIP.

[san]

Уточню, что задача Михаила относится к прошивке "для института", которая у нас называется "VE-02-no-PoE"

[alx]

Replying to mixyil1.1:

Это рассматривается как недокументированные возможности шлюза, которых не должно быть.

Я согласен с тем, что недокументированных возможности - это нехорошо. В идеале их не должно быть.

В связи с чем институт "Агат" просит рассмотреть возможность ревизии исходного кода на предмет удаления кода относящегося к тем возможностям платы VE-01, которые не используются в MC04-DSL-VIP.

Но вызывает удивление предложенное решение.

Когда блок MC04-DSL-VIP разрабатывался, его назначением было решение очень узкой задачи - подключения домика оператора ГРС к системе диспетчерской связи. Для ответа на вызов диспетчера оператору не требовались ДВО, в том числе создание конференции, поэтому соответствующая настройка не была сделана в веб-интерфейсе блока и не описана в документации. Однако сейчас блок MC04-DSL-VIP позиционируется как более универсальное решение. Поэтому сейчас функции ДВО вполне могут оказаться полезны и востребованы в некоторых применениях блока, и удаление их из кода я считаю неправильным. Мне кажется, что если возможность уже реализована в коде, то более конструктивным решением будет документировать имеющуюся возможность, нежели уничтожать ее.

Кроме этого (и это, пожалуй, главная причина), плата VE-02, используемая в блоке MC-04-DSL-VIP, имеет два исполнения: 0 и 1. Плата VE-02 исполнения 1 используется в блоке MC04-DSL-3U, где используются все функции, в том числе и управление конференциями. Оба исполнения платы управляются одним и тем же кодом, поэтому удалить этот код без ущерба для функционала MC04-DSL-3U изделий нельзя.

[alx]

Replying to san:

Уточню, что задача Михаила относится к прошивке "для института", которая у нас называется "VE-02-no-PoE"

Саша, эта прошивка - штатная прошивка платы, в которой закомментированы лишь несколько строк - обращения к контроллеру PoE, который не установлен на плату для института, и обращение к которому приводило к "подвисанию" и перезагрузке. Одно дело - закомментировать несколько однотипных строк, совсем другое - "выпиливать" из кода некий функционал (который, в отличие от PoE, никому не мешает). Несопостовимая трудоемкость.

[san]

штатная прошивка платы, в которой закомментированы лишь..

Тем не менее это отдельная ветка ПО.

Когда блок MC04-DSL-VIP разрабатывался, его назначением было решение очень узкой задачи - подключения домика оператора ГРС к системе диспетчерской связи

Директор принял решение использовать блок VIP для проекта "института", насколько я его понял он хочет, чтобы для этого проекта была отдельная прошивка содержащая только функции необходимые для проекта, но не буду говорить за него, а лучше добавлю в копию.

[alx]

Replying to san:

Директор принял решение использовать блок VIP для проекта "института", насколько я его понял он хочет, чтобы для этого проекта была отдельная прошивка содержащая только функции необходимые для проекта, но не буду говорить за него, а лучше добавлю в копию.

А насколько я помню, сначала директор вообще не планировал делать для этой версии специальный софт, а лишь модифицировать плату (демонтировать с нее ненужные компоненты). Но когда выяснилось, что после демонтажа софт перестал работать, мне было предложено "по-быстрому" закомментировать обращения к контроллеру. Я это сделал, обновил файл в платах, убедился, что они заработали, и удалил исходники. Когда через несколько месяцев поступило указание сделать еще пару плат, я снова закомментарил те же строки, и на этот раз сохранил в репозиторий svn (на всякий случай, вдруг третий раз потребуется). Вот так и появилась ветка, о которой ты говоришь. :)