Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

System test broker with crash #754

Merged
merged 4 commits into from
Dec 9, 2020
Merged

System test broker with crash #754

merged 4 commits into from
Dec 9, 2020

Commits on Dec 9, 2020

  1. System test broker with crash

    @kleunen @redboltz
    kleunen authored and redboltz committed Dec 9, 2020
    Configuration menu
    Copy the full SHA
    ae6a066 View commit details
    Browse the repository at this point in the history

  2. Removed post() call from packet parse process. 

    It can avoid event handler call after close.
It requires a little big stack.
If the packet contains N properties, then call stack complexity is O(N).
N times recursive call of property parsing function.
I'm not sure it is acceptable.
    @redboltz
    redboltz committed Dec 9, 2020
    Configuration menu
    Copy the full SHA
    f8395a4 View commit details
    Browse the repository at this point in the history

  3. Fixed test shutdown code. 

    Renamed test.
    @redboltz
    redboltz committed Dec 9, 2020
    Configuration menu
    Copy the full SHA
    cf6e816 View commit details
    Browse the repository at this point in the history

  4. Fixed close or error detection logic. 

    Even if async_read()'s error_code is not error, the connection has
already been disconnected then regard to error.
    @redboltz
    redboltz committed Dec 9, 2020
    Configuration menu
    Copy the full SHA
    77fe3fc View commit details
    Browse the repository at this point in the history