Re: [multi] Formal Review Begins
Seth via Boost <[email protected]>
| Newsgroups | gmane.comp.lib.boost.devel |
|---|---|
| Message-ID | <[email protected]> |
On Sat, Mar 7, 2026, at 10:41 PM, Andrzej Krzemienski via Boost wrote:
> multi::array<int, 2> a({5, 2}, 5); // start with 5's
> std::ranges::fill(a.elements(), 6); // fill with 6's
> multi::array<int, 2> b = restricted([](int i, int j) { return i * j * 10;
> }, multi::extensions_t{5, 2}); // start with index-dependent values
>
> Regards,
> &rzej;
Not a review, but I have seen `multi::extensions_t` and `a.extensions()` and wonder whether any reviewers have already mentioned that that name seems off?
I'm not a native speaker, but with a bit of math background and experience with other [multi-dimensional] libraries I expect these to be extent, extents, extents_t (see e.g. https://en.cppreference.com/w/cpp/types/extent.html, https://www.boost.org/doc/libs/latest/libs/multi_array/doc/user.html#sec_dimensions)
It would be nice to adhere to common jargon instead potentially creating friction/confusion for users coming to Boost.Multi.
Just a note in case you want to consider it.
Seth
_______________________________________________
Boost mailing list -- [email protected]
To unsubscribe send an email to [email protected]
https://lists.boost.org/mailman3/lists/boost.lists.boost.org/
Archived at: https://lists.boost.org/archives/list/[email protected]/message/GCA36HGM4RZU7WWHLQECTGVCISTROPNN/