LINUX.ORG.RU
ФорумTalks

Мелочи обратной совместимости

 , ,


0

1

Запустил mpv с субтитрами и обратил внимание на предупреждение:

Warning: option --sub was replaced with --sub-file and might be removed in the future.

Хорошо, что предупредили, что возможно заменят ключ, но зачем его вообще менять, чешется что ли? По «фен-шую» раздать названия захотелось? Давным-давно было sub, ещё в первом mplayer было. А потом после замены где-то возможно даже в прошивке какого медиаплеера отвалятся субтитры, если сборщик не подумает провести полный тест всей функциональности.

Дело не в mpv, просто такие замены время от время случаются в самых разных программах и что хуже API.

Но может я слишком консервативным стал с возрастом и уже не понимаю всей сермяжной правды таких изменений?

У меня на прошлой работе был такой программист: его позвали отрефакторить ядро системы оркестрации, а он занялся переименовыванием функций в соответствии со своим чувством прекрасного.

leave ★★★★★ ()

mpv появился в том числе и для того, чтобы почистить старые ключи и опции, и подобных изменений там, в общем-то, дохера. Если мне не изменяет память, про обратную совместимость никто не говорил. Ну а если тебя это не устраивает, то тебе никто не мешает пользоваться старым-добрым mplayer, в котором ничего подобного нет.

theNamelessOne ★★★★★ ()
Ответ на: комментарий от theNamelessOne

mpv появился в том числе и для того, чтобы почистить старые ключи и опции, и подобных изменений там, в общем-то, дохера. Если мне не изменяет память, про обратную совместимость никто не говорил.

Не знал, думал просто форк проекта.

Ну а если тебя это не устраивает, то тебе никто не мешает пользоваться старым-добрым mplayer, в котором ничего подобного нет.

На mpv перешёл из-за глюков mplayer. Я люблю иногда фильм в интересных сценах несколько раз быстро погонять туда сюда, обычно при этом непрерывно нажимая клавиши курсора и стопя/пуская по чуть-чуть пробелом. mplayer стал часто зависать при этом, mpv нормально отрабатывает.

anonymous_incognito ★★★★★ ()
Последнее исправление: anonymous_incognito (всего исправлений: 1)

Ну епт, возможно они планируют добавить опции для сабов типа --sub-encodig

upd. а так и есть:


    --ass → --sub-ass
    --autosub → --sub-auto
    --autosub-match → --sub-auto-match
    --sub → --sub-file
    --subcp → --sub-codepage
    --subfps → --sub-fps

так что все правильно, а то шо в мплеере - песец какойто был, читать маны и раскуривать что как управляется

Deleted ()
Последнее исправление: Deleted (всего исправлений: 1)

Править ошибки молодости — достойное дело.

kirk_johnson ★☆ ()

ещё в первом mplayer было.

Так они специально не тащат совместимость с «ещё первым mplayer» ради чистоты кода.

INFOMAN ★★★★★ ()
Ответ на: комментарий от leave

отрефакторить ядро системы

занялся переименовыванием функций в соответствии со своим чувством прекрасного

Простите, но это одно и то же (по определению). Что не понравилося, типо много переименовал что ли или что? "Зачем звала дура, и так всё работает" (c) Или у системных администраторов под "рефакторингом" что-то своё понимается, чтобы лишнее слово не тратить.

*мимо не тот программист*

d_a ★★★★★ ()

они lirc выпилили, а ты про какой-то --sub жалуешься...

dikiy ★★☆☆☆ ()
Ответ на: комментарий от anonymous_incognito

Я люблю иногда фильм в интересных сценах несколько раз быстро погонять туда сюда, обычно при этом непрерывно нажимая клавиши курсора и стопя/пуская по чуть-чуть пробелом. mplayer стал часто зависать при этом, mpv нормально отрабатывает.

котрой для себя кнопку "."

dikiy ★★☆☆☆ ()

ты даже не представляешь как меня эта моральная мастурбация с названиями функций достала в LLVM. некоторые уже по 2-3 раза переиначили за полгода

upcFrost ★★★★★ ()

По «фен-шую» раздать названия захотелось?

А этого вполне достаточно. Судя по документации, был вообще кромешный п**ц, я бы даже совместимость с этим не оставлял.

* There is a new command line syntax, which is generally preferred over the old
  syntax. ``-optname optvalue`` becomes ``--optname=optvalue``.

  The old syntax will not be removed. However, the new syntax is mentioned in
  all documentation and so on, and unlike the old syntax is not ambiguous,
  so it is a good thing to know about this change.

...

    ``-sub-fuzziness``          ``--sub-auto``
    ``-sub-text-*``             ``--sub-*``
    ``-sub``                    ``--sub-file``
    ``-subcp``                  ``--sub-codepage``
    ``-subdelay``               ``--sub-delay``
    ``-subfile``                ``--sub-file``
    ``-subfont-*``              ``--sub-*``, ``--osd-*``
    ``-subfont-text-scale``     ``--sub-scale``
    ``-subfont``                ``--sub-font``
    ``-subfps``                 ``--sub-fps``
    ``-subpos``                 ``--sub-pos``
    ``-vobsub``                 ``--sub-file`` (pass the .idx file)
slovazap ★★★★★ ()
Ответ на: комментарий от d_a

Там внутри ад и погибель, костыль на костыле, как результат - тормоза. Человека с тайтлом лида зовут причесать это дерьмо, он переименовывает функцию game() в _game().

leave ★★★★★ ()
Ответ на: комментарий от imul

Ну работает до сих пор вроде. Все так же уныло, но работает. По уму, сейчас уже проще с нуля сделать новую систему и для новых площадок использовать ее, но есть куча нюансов, из-за которых не получится отвязаться от старой системы работы с конфигами.

leave ★★★★★ ()
Ответ на: комментарий от deep-purple

Была переменная game (локальная для функции), и функция game(). Стало красиво :)

leave ★★★★★ ()
Вы не можете добавлять комментарии в эту тему. Тема перемещена в архив.