container_common_test.: out of bounds warning on GCC 14 about functionalplus HOT 5 CLOSED

Ok, I pushed the change (including a comment). ✔️

from functionalplus.

Fixed by #302

from functionalplus.

Oh, what an interesting false positive (I guess), thanks!

I'll see if I can build a workaround for FunctionalPlus, and maybe even a minimal example to reproduce the problem.

from functionalplus.

I've experimented locally, and replacing return xs; with return Container({xs.front()}); seems to work. It's not beautiful, but it should also not be a significant performance penalty. Shall we use this? And would you like to push it to your branch, or shall I open a new one?

from functionalplus.

I've experimented locally, and replacing return xs; with return Container({xs.front()}); seems to work.

Interesting 🤔.

Shall we use this? And would you like to push it to your branch, or shall I open a new one?

If it works, why not? :-) There' should be a comment documenting the reason though.

And would you like to push it to your branch, or shall I open a new one?

You should have write access to me PR, feel free to add commits directly. 👍 Otherwise, what suits you best.

from functionalplus.