Альтернативное извлечение порта и бита из пина #7
Conversation
| #define F6 30 | ||
| #define F7 31 | ||
|
|
||
| #define fetch_port_from_pin( x ) (0b11000 & x) |
There was a problem hiding this comment.
fetch — это куда-нибудь сходить и вернуться с чем-нибудь, комплексное действие. не очень подходит для такой простой и частоиспользуемой операции. думаю, имеет смысл сразу замещать существующую реализацию port_from_pin, всё равно это — отдельная ветка пулл-реквеста
|
мержить этот пулл-реквест? |
|
нет, зачем же. по феншую его положено вмержить, когда реализация полностью написана и все достигли соглашения, что всё ок. |
А по факту - это единственное, с чем мы работаем, и странно, что это не мастер. |
|
СТОП! Мы не там обсуждаем пулреквест. Это пул моего патча, который я действительно не хочу никуда пулять, а просто размышляю, в мой же бранч. Предлагаю так: я чищу тулчейн от моих оберток и мержу это в мастер. По реализациям функций уже работаем по-человечески. (Если честно, до сегодняшнего дня не просекал что тут зачем, а теперь готов работать нормально) |
Вообще я и сам имею право мержить, я ж коллаборатор :) Но этот патч - просто мысли в слух, и явно кривые |
Альтернативное извлечение порта и бита из пина Мерж только потому, что я не знаю, как дальше жить
|
так а нафиг смёржил-то? |
|
да всё норм |
No description provided.