sadr0b0t / stepper_h

Stepper motor control library for ChipKIT/Arduino
GNU Lesser General Public License v3.0
3 stars 1 forks source link

Рефактор API stepper: stepper_info_t отправить полем в stepper #20

Closed sadr0b0t closed 7 years ago

sadr0b0t commented 7 years ago

Структура stepper_info_t содержит динамическую информацию об одном моторе:

Сейчас ее предлагается отправлять последним необязательным параметром в prepare_xxx_steps, чтобы иметь дополнительную возможность отслеживать ошибки (типа это второстепенные вещи, которые не хотелось пихать в основной API), хотя по факту большой пользы в этом нет (места они занимаю не много, а коды ошибок очень даже нормальный функционал в основном API тоже).

Плюс, структура stepper и так уже содержит помимо исходных статичных полей (настройки мотора) динамическое поле current_pos. Будет логично объединить всю динамическую информацию по мотору в одной подструктуре или подразделе структуры stepper.

Короче:

sadr0b0t commented 7 years ago

Всё есть (просто замержил все поля из stepper_info_t в структуру stepper, так удобнее) https://github.com/1i7/stepper_h/commit/4f53763bc491ed19e1054888173f20abb8e2bd25

кроме последнего:

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

перенес в новый тикет https://github.com/1i7/stepper_h/issues/21