SHS/pip
4
3
Fork 0
Code Issues 16 Pull Requests 2 Actions Releases Wiki Activity

PICout переделать API для работы с внешними буферами #103

New Issue
Closed
opened 2022-08-04 16:05:31 +03:00 by andrey · 4 comments
andrey commented 2022-08-04 16:05:31 +03:00
Owner
Copy Link
Copy Source

сейчас есть набор функций

bool setBufferActive(bool on, bool clear = false);
bool isBufferActive();
void clearBuffer();
bool setOutputDevice(OutputDevice d, bool on = true);
void setOutputDevices(OutputDevices d);
bool isOutputDeviceActive(OutputDevice d);

по названию функций и куче аргументов тяжело разобраться как этим пользоваться, и непонятно зачем одно и тоже делать разными функциями

сейчас есть набор функций ``` bool setBufferActive(bool on, bool clear = false); bool isBufferActive(); void clearBuffer(); bool setOutputDevice(OutputDevice d, bool on = true); void setOutputDevices(OutputDevices d); bool isOutputDeviceActive(OutputDevice d); ``` по названию функций и куче аргументов тяжело разобраться как этим пользоваться, и непонятно зачем одно и тоже делать разными функциями
andrey commented 2022-08-04 16:09:36 +03:00
Author
Owner
Copy Link
Copy Source

Из за этого неясного API уже сейчас в коде есть дублирование вызова функций
https://git.shs.tools/SHS/pip/src/branch/master/libs/console/piscreentiles.cpp#L540

Из за этого неясного API уже сейчас в коде есть дублирование вызова функций https://git.shs.tools/SHS/pip/src/branch/master/libs/console/piscreentiles.cpp#L540
andrey commented 2022-08-04 16:16:45 +03:00
Author
Owner
Copy Link
Copy Source

Ещё непонятно чем принципиально отличается PICout(buffer) и PICout::setBufferActive

Ещё непонятно чем принципиально отличается PICout(buffer) и PICout::setBufferActive
andrey commented 2022-08-04 16:18:40 +03:00
Author
Owner
Copy Link
Copy Source

функция setControl(PICoutManipulators::PICoutControls c, bool save = false) совершенно лишняя

функция setControl(PICoutManipulators::PICoutControls c, bool save = false) совершенно лишняя
andrey commented 2022-08-04 16:23:21 +03:00
Author
Owner
Copy Link
Copy Source

enum OutputDevice и флаги OutputDevices убрать в private, и заменить функциями по типу
enableBuffer, enableSTDout и т.п.

enum OutputDevice и флаги OutputDevices убрать в private, и заменить функциями по типу enableBuffer, enableSTDout и т.п.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
Баг
Косяк
 В работе
Ставим как приступили к выполнению
 Назначена
Ставим когда назначен исполнитель
 Отложено
Выполнение задачи отложено или ожидает других
 Сделано
Ставим когда выполнили задачу, и ожидаем одобрения
 Тестируется
Ставим когда отправили задачу на тестирование
 Требует обсуждения
Задача вызывает трудности или вопросы
 Фича
Новый функционал
No labels
Milestone
No items
No Milestone
Assignees
Clear assignees
andrey (Бычков Андрей) drone Gama (Дима Федоренко) jenkins (jenkins) maakshishov (Максим Шишов) peri4 (Пелипенко Иван) tambaru
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SHS/pip#103