Re: [PATCH] test-flow-perf: Enable to build on Windows

[Andre Muezerie <andremue@linux.microsoft.com>]

On Fri, Aug 29, 2025 at 03:04:10PM +0200, David Marchand wrote:
> Hello,
> 
> On Tue, 5 Aug 2025 at 22:00, Andre Muezerie
> <andremue@linux.microsoft.com> wrote:
> >
> > This patch fixes some issues which were preventing this test to be
> > built on Windows:
> >
> > - Remove VLAs (not supported by msvc).
> > - Replace strsep() (which is not natively available on Windows)
> >   with strtok_r().
> > - Remove the "thousands" separator from printf() calls as it is
> >   not available on Windows.
> 
> Can we use rte_size_to_str() instead?
> 

I considered doing that, but I thought that since the app is printing deltas it
could be better to print the exact numbers instead of some approximation like
"1.02 k" which could possibly make debugging harder.

If you strongly believe that rte_size_to_str() should indeed be used here I can
make the change.

What are your thoughts about this?

Andre Muezerie

> > - Include the test in the Windows build.
> >
> > Signed-off-by: Andre Muezerie <andremue@linux.microsoft.com>
> 
> Otherwise it lgtm.
> 
> 
> -- 
> David Marchand