feat: макрос uformat #21

Merged
DarkCat09 merged 2 commits from uformat into master 2025-01-10 16:20:30 +03:00
Owner
No description provided.
DarkCat09 added 2 commits 2025-01-09 18:19:29 +03:00
requested review from nm17 2025-01-09 18:19:37 +03:00
Author
Owner

Сделал .unwrap в макросе, то есть ufmt перестал быть no-panic. Убрать и везде делать uformat!(...).expect("ну чё-т ошибка") или так и оставить, консистенто с format?

Сделал `.unwrap` в макросе, то есть ufmt перестал быть no-panic. Убрать и везде делать `uformat!(...).expect("ну чё-т ошибка")` или так и оставить, консистенто с format?
Author
Owner

Ну и не трогал где редис, там ты HeaplessString юзаешь вместо String:

let mut key_buffer = HeaplessString::<{ 7 + 13 }>::new();
uwrite!(key_buffer, "apikey_{}", api_key).expect("TODO"); // TODO: Error handling

Ну и не трогал где редис, там ты HeaplessString юзаешь вместо String: https://git.dc09.ru/iotishnik/server/src/commit/5c4d70b46e3f036e62b0b838fd0697a0136f7fd0/src/web_server/utils/redis.rs#L36-L37
Author
Owner

Могу сделать макрос uformat!<Type>(), чтоб можно было не только в String писать

Могу сделать макрос `uformat!<Type>()`, чтоб можно было не только в String писать
Author
Owner

(из телеграма)

Если unwrap на что-то почти не возможное - то оставь

Я поэтому и спрашиваю -- даже не пытался изучать, в каком случае format! выдаёт ошибку, но как будто ни разу не видел такого кейса

(из телеграма) > Если unwrap на что-то почти не возможное - то оставь Я поэтому и спрашиваю -- даже не пытался изучать, в каком случае format! выдаёт ошибку, но как будто ни разу не видел такого кейса
nm17 approved these changes 2025-01-10 16:20:15 +03:00
DarkCat09 merged commit c7ec61994b into master 2025-01-10 16:20:30 +03:00
DarkCat09 deleted branch uformat 2025-01-10 16:20:35 +03:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Reference: iotishnik/server#21
No description provided.