>>>>> "vshcherb" == vshcherb  <notifications@github.com> writes:

    vshcherb> In java/net/osmand/plus/routing/VoiceRouter.java:
    >> @@ -176,7 +176,7 @@ protected void makeUTStatus() { * Updates
    >> status of voice guidance * @param currentLocation */ - protected
    >> void updateStatus(Location currentLocation) { + protected void
    >> updateStatus(Location currentLocation, boolean scheduledCall) {

    vshcherb> Я предлагаю назвать переменную repeatPrevious и если надо
    vshcherb> обрабатывать здесь.

В смысле, второй параметр updateStatus()? Не возражаю, хотя, строго
говоря, это ведь получается не повторение предыдущей инструкции, а
выработка по требованию актуальной на данный момент времени.

    vshcherb> На самом деле функция "Повторить" имеет большое значение
    vshcherb> не только для accessiblity,

Честно говоря, я тоже так думаю, потому удивлен был несколько, что в
программе это не предусмотрено.

    vshcherb> поэтому предлагаю сделать это очевидным параметром в
    vshcherb> функции. Возможно функцию вообще стоит переименовать в
    vshcherb> announceVoiceDirection(Location location, boolean
    vshcherb> announceOnlyIfChanged)

Разумеется, никаких возражений с моей стороны. Я просто действовал, как
обычно, по принципу наименьшего вмешательства.