Skip to content
This repository was archived by the owner on May 31, 2025. It is now read-only.

[Melodic Backport] set call_finished_ with true for each call inside callFinished (#2074)#2080

Closed
iuhilnehc-ynos wants to merge 1 commit intoros:melodic-develfrom
iuhilnehc-ynos:topic-backport-2074-for-melodic
Closed

[Melodic Backport] set call_finished_ with true for each call inside callFinished (#2074)#2080
iuhilnehc-ynos wants to merge 1 commit intoros:melodic-develfrom
iuhilnehc-ynos:topic-backport-2074-for-melodic

Conversation

@iuhilnehc-ynos
Copy link
Copy Markdown
Contributor

  • set call_finished_ with true for each call inside callFinished

and also set current_call flags when failure happens

Signed-off-by: Chen Lihui lihui.chen@sony.com

  • Update based on review

Co-authored-by: Tomoya.Fujita Tomoya.Fujita@sony.com
Signed-off-by: Chen Lihui lihui.chen@sony.com

  • set the flag with true again even if it is true already

Signed-off-by: Chen Lihui lihui.chen@sony.com

Co-authored-by: Tomoya.Fujita Tomoya.Fujita@sony.com

)

* set call_finished_ with true for each call inside callFinished

and also set current_call flags when failure happens

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

* Update based on review

Co-authored-by: Tomoya.Fujita <Tomoya.Fujita@sony.com>
Signed-off-by: Chen Lihui <lihui.chen@sony.com>

* set the flag with true again even if it is true already

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

Co-authored-by: Tomoya.Fujita <Tomoya.Fujita@sony.com>
@fujitatomoya
Copy link
Copy Markdown
Contributor

@iuhilnehc-ynos

i think this wont be necessary, see #1496. i guess that you can close this PR.

@dirk-thomas
Copy link
Copy Markdown
Member

Closing in favor of #2074 (comment).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants