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

Keep custom allocator in publisher and subscription options alive. #1647

Merged
merged 4 commits into from
May 4, 2021
Merged

Keep custom allocator in publisher and subscription options alive. #1647

merged 4 commits into from
May 4, 2021

Commits on Apr 29, 2021

  1. Keep custom allocator in publisher and subscription options alive. 

    Also, enforce an allocator bound to void is used to avoid surprises.

Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
    @hidmic
    hidmic committed Apr 29, 2021
    Configuration menu
    Copy the full SHA
    0f768bb View commit details
    Browse the repository at this point in the history

  2. Avoid sizeof(void) in MyAllocator implementation. 

    Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
    @hidmic
    hidmic committed Apr 29, 2021
    Configuration menu
    Copy the full SHA
    f4dcd4e View commit details
    Browse the repository at this point in the history

Commits on Apr 30, 2021

  1. Address peer review comment 

    Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
    @ivanpauno
    ivanpauno committed Apr 30, 2021
    Configuration menu
    Copy the full SHA
    6ff1246 View commit details
    Browse the repository at this point in the history

Commits on May 4, 2021

  1. Use a lazely initialized private field when 'allocator' is not initia… 

    …lized

Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
    @ivanpauno
    ivanpauno committed May 4, 2021
    Configuration menu
    Copy the full SHA
    fe13993 View commit details
    Browse the repository at this point in the history