[PATCH i-g-t v3 2/6] runner: Make it easier to extend argv
Lucas De Marchi
lucas.demarchi at intel.com
Mon Aug 12 13:54:19 UTC 2024
On Thu, Jul 25, 2024 at 11:19:35AM GMT, Gustavo Sousa wrote:
>In an upcoming change, we will be adding the option to forward the
>--hook option to the test cases, which will require updating
>execute_test_process() to add the option when asked by the user.
>
>The current implementation makes that task not quite straightforward:
>filling of argv is already dependent on stuff like entry->subtest_count
>and dynbegin; if we want to keep on using constant indices, we would
>need several conditional branches for adding arguments for --hook.
>
>Let us change the current implementation to use a dynamic vector, to
>make it easier to extend argv with more stuff as needed.
>
>v2:
> - Squash the logic from patch "runner: Use dynamic vector for test
> argv" directly instead of using the original logic, which used a
> statically sized array. (Lucas)
>
>Signed-off-by: Gustavo Sousa <gustavo.sousa at intel.com>
Reviewed-by: Lucas De Marchi <lucas.demarchi at intel.com>
Lucas De Marchi
More information about the igt-dev
mailing list