Skip to content
Snippets Groups Projects

Refactor tarantool_main macro

Merged Yaroslav Dynnikov requested to merge raft-join-05-tarantool_main-macro into master

@gmoshkin · 1 week ago

у макроса tarantool_main была такая фича, что тип cb_data фиксировался тем, что мы замыкание передавали, которое в колбеке вызывается. При чём колбек можно передать в extern "C" функцию только через макрос. Чем дальше находятся места, где данные в бокс заворачиваются и где они разворачиваются, тем проще допустить ошибку. Я предлагаю сделать минимальный макрос без разветвлений, который будет принимать замыкание, которое выполняется в колбеке.

!71 (comment 5482)

Edited by Yaroslav Dynnikov

Merge request reports

Merge request pipeline #3970 passed

Merge request pipeline passed for 2873f401

Merged by Yaroslav DynnikovYaroslav Dynnikov 2 years ago (Apr 25, 2022 3:11pm UTC)

Loading

Pipeline #3971 passed

Pipeline passed for 2873f401 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
Please register or sign in to reply
Loading