Skip to content

Allow array types and values of arbitrary size#791

Merged
bocchino merged 22 commits intomainfrom
issue-694-array-sizes
Sep 2, 2025
Merged

Allow array types and values of arbitrary size#791
bocchino merged 22 commits intomainfrom
issue-694-array-sizes

Conversation

@bocchino
Copy link
Collaborator

@bocchino bocchino commented Aug 21, 2025

Closes #694.

Procedure:

Use FwSizeType instead of U32 for indices
Add operator= with initializer list
Use the single-value constructor when all the element values
are the same
Add initializer list constructor
Use initializer list for array values
Remove multi-element constructor
Use initializer list constructor instead
Pass initializer list by reference in constructor

This seems to help the compiler disambiguate constructor arguments
in some cases
Allow large-size arrays
Add tests
Make single element constructor explicit
@bocchino bocchino requested a review from Kronos3 August 21, 2025 15:24
@bocchino bocchino added the fprime-fpp tandem F Prime and FPP tandem development label Aug 21, 2025
Copy link
Collaborator

@Kronos3 Kronos3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good!

@thomas-bc
Copy link
Contributor

nasa/fprime#4073 is merged !

@bocchino bocchino merged commit 72d368f into main Sep 2, 2025
30 checks passed
@bocchino bocchino deleted the issue-694-array-sizes branch September 2, 2025 15:25
@Kronos3 Kronos3 mentioned this pull request Sep 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fprime-fpp tandem F Prime and FPP tandem development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Do not limit FPP array sizes to 256 elements

3 participants