<div dir="auto">I'd add review-by tags or tested-by tags but I'm also Co-Author, would feel wrong lol.<div dir="auto"><br></div><div dir="auto">Just a reminder about this patch set.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed 20 Jul 2022, 15:54 Eric Curtin, <<a href="mailto:ecurtin@redhat.com">ecurtin@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, 20 Jul 2022 at 14:04, Laurent Pinchart<br>
<<a href="mailto:laurent.pinchart@ideasonboard.com" target="_blank" rel="noreferrer">laurent.pinchart@ideasonboard.com</a>> wrote:<br>
><br>
> Hello,<br>
><br>
> This patch series bundles a new version of Eric's SDL2_image replacement<br>
> with an additional cleanup for the cam SDL support. I've numbered it v5<br>
> as Eric's patch (now 1/2) was v4, but patch 2/2 is new.<br>
><br>
> The changes in 1/2 just incorporates my review comments, I've decided to<br>
> send the result as I had already made the changes locally to test the<br>
> review recommedations.<br>
><br>
> Eric, please feel free to disagree with any of the changes :-)<br>
> Otherwise, if you're OK with them, I'll apply the series.<br>
<br>
I tested and reviewed the patches, all good on my end!<br>
<br>
><br>
> Eric Curtin (1):<br>
>   cam: sdl_sink: Use libjpeg over SDL2_image<br>
><br>
> Laurent Pinchart (1):<br>
>   cam: sdl_sink: Pass a Span<const uint8_t> to SDLTexture::update()<br>
><br>
>  README.rst                   |  2 +-<br>
>  src/cam/meson.build          |  8 ++--<br>
>  src/cam/sdl_sink.cpp         |  4 +-<br>
>  src/cam/sdl_texture.h        |  2 +-<br>
>  src/cam/sdl_texture_mjpg.cpp | 72 ++++++++++++++++++++++++++++++++----<br>
>  src/cam/sdl_texture_mjpg.h   |  8 +++-<br>
>  src/cam/sdl_texture_yuyv.cpp |  2 +-<br>
>  src/cam/sdl_texture_yuyv.h   |  2 +-<br>
>  8 files changed, 82 insertions(+), 18 deletions(-)<br>
><br>
><br>
> base-commit: 8b02645845bf404fba83910818b285b652f14f89<br>
> --<br>
> Regards,<br>
><br>
> Laurent Pinchart<br>
><br>
</blockquote></div>