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

issue: 3928407 Remove VLAs #142

Open
wants to merge 2 commits into
base: vNext
Choose a base branch
from
Open

issue: 3928407 Remove VLAs #142

wants to merge 2 commits into from

Commits on Oct 10, 2024

  1. issue: 3928407 Refactor net_device_val::get_up_and_active_slaves() 

    Remove VLAs and simplify net_device_val::get_up_and_active_slaves().

Signed-off-by: Dmytro Podgornyi <[email protected]>
    @pasis
    pasis committed Oct 10, 2024
    Configuration menu
    Copy the full SHA
    dea4aa1 View commit details
    Browse the repository at this point in the history

  2. issue: 3928407 Remove VLAs to fix clang-18 build 

    clang-18 generates a warning that VLA is a clang extension. Allocate
the arrays on heap instead of stack.

Signed-off-by: Dmytro Podgornyi <[email protected]>
    @pasis
    pasis committed Oct 10, 2024
    Configuration menu
    Copy the full SHA
    55214ea View commit details
    Browse the repository at this point in the history