On Fri, May 16, 2025 at 4:44 PM Thomas Monjalon <tho...@monjalon.net> wrote: > > 16/05/2025 14:50, Andre Muezerie: > > On Fri, May 16, 2025 at 10:01:42AM +0200, David Marchand wrote: > > > On Fri, May 9, 2025 at 11:52 PM Andre Muezerie > > > <andre...@linux.microsoft.com> wrote: > > > > > > > > Now that mbuf is compiling on Windows, this library can be enabled as > > > > well. > > > > > > > > Signed-off-by: Andre Muezerie <andre...@linux.microsoft.com> > > > > > > The test-regex app can probably be compiled too. > > > Yet.. why do you want to enable this library on Windows? > > > > > > Support for this device class is defacto deprecated, as no vendor is > > > providing recent hardware implementing this feature. > > > > > > Also note that it was not built with clang on Windows so far (I did > > > not double check this assertion, I just based on the fact this patch > > > removes a is_windows, and not a is_ms_compiler). > > > > > > > > > > I was not aware this code was deprecated. You're right that it was not > > being compiled with Clang. This was just part of the effort to increase > > code compatibility on Windows. > > > > I see no harm in enabling it, but at the same time, there's no clear > > benefit either. I'll let you decide what to do with this. > > The benefit is to remove special handling for Windows, > so I vote for enabling it. > There was no regex development recently but we never know > how it could be used in future :)
Well yes, there is no activity so adding this for Windows will probably not add much maintenance work.. -- David Marchand
