• X xray-16
  • Информация о проекте
    • Информация о проекте
    • Активность
    • Метки
    • Участники
  • Репозиторий
    • Репозиторий
    • Файлы
    • Коммиты
    • Ветки
    • Теги
    • Участники
    • Диаграмма
    • Сравнение
  • Задачи 216
    • Задачи 216
    • Список
    • Доски
    • Спринты
  • Запросы на слияние 31
    • Запросы на слияние 31
  • CI/CD
    • CI/CD
    • Конвейеры
    • Задания
    • Расписания
  • Развертывания
    • Развертывания
    • Окружения
    • Релизы
  • Пакеты и реестры
    • Пакеты и реестры
    • Реестр пакетов
    • Реестр контейнеров
  • Мониторинг
    • Мониторинг
    • Инциденты
  • Аналитика
    • Аналитика
    • Поток ценности
    • CI/CD
    • Репозиторий
  • Wiki
    • Wiki
  • Сниппеты
    • Сниппеты
  • Активность
  • Диаграмма
  • Создать новую задачу
  • Задания
  • Коммиты
  • Доски с задачами
Свернуть панель
  • OpenXRay
  • xray-16
  • Запросы на слияние
  • !846

command line options, started transition (#570)

  • Ревью изменений

  • Скачать
  • Почтовые патчи
  • Простое отличие
Закрыто xottab-duty запросил слияние github/fork/jjdredd/jjdredd-wip-570-proposal в dev Июл 12, 2021
  • Обзор 22
  • Коммиты 18
  • Конвейеры 0
  • Изменения 24

Created by: jjdredd

I wish to discuss these changes. New system for handling command line options.

While transitioning to the news system, I noticed that this approach is very inconvenient when adding command line options that are used across different subsystems. One has to define a public variable and declare it external in other places where it's used, keeping in mind that the unit where external declaration is used must be compiled after the unit where the variable is defined. This is not very readable and extendable, although not many options need to be used across different modules.

Another option is to have a public variable containing a map of command line options and access it like so: command_line_options_map.getvalue("-myoption"). This could lead to some overhead.

This PR is just for discussion.

Ответственный
Назначить
Проверяющие
Запросить ревью
Оценка трудозатрат
Исходная ветка: github/fork/jjdredd/jjdredd-wip-570-proposal