[libcamera-devel] [PATCH 13/16] libcamera/base: Move span to base library

[Hirokazu Honda]

Hi Kieran,

On Fri, Jun 25, 2021 at 7:59 PM Kieran Bingham
<kieran.bingham at ideasonboard.com> wrote:
>
> Hi Hiro,
>
> On 25/06/2021 10:39, Kieran Bingham wrote:
> >
> >>
> >> Isn't span not one which should not be exposed to public?
> >
> > Oh that's interesting - I thought span was used in the public API
> > somewhere, but now I can't see it - I'll check and move to private if it
> > can go there.
>
> We use spans for controls, which form part of the public-api.

Ah, I see. Acknowledged.

Reviewed-by: Hirokazu Honda <hiroh at chromium.org>
>
> So base/span.h is currently public, and will stay there (until C++20? :D)
>
> --
> Regards
> --
> Kieran