-
Notifications
You must be signed in to change notification settings - Fork 16
Conversation
Use |
tangle_accelerator_core_api.c
Outdated
} | ||
|
||
ta_generate_address_res_serialize(&json_result, res); | ||
if (ret) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will ret
be changed in previous line?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes! Fixed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If goto
is used right after if (ret)
, this function would look lighter.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also the last check is not needed I think
1951a67
to
f9a0c15
Compare
tangle_accelerator_core_api.c
Outdated
|
||
ret = ta_generate_address_res_serialize(&json_result, res); | ||
if (ret) { | ||
goto done; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This goto
is not necessary here.
Rebasing is required. |
69e62f7
to
a6c42e1
Compare
Close #6