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

New Issue

andrey · 2022-08-04T16:05:31+03:00

andrey commented

2022-08-04 16:05:31 +03:00

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

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

Из за этого неясного 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

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

andrey commented

2022-08-04 16:18:40 +03:00

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

andrey commented

2022-08-04 16:23:21 +03:00

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

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

andrey closed this issue

2022-08-10 15:10:32 +03:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: SHS/pip#103