Skip to content

Make default string size configurable in the model, use in cpp and dictionary codegen#909

Closed
jwest115 wants to merge 4 commits intonasa:mainfrom
jwest115:default-string-size
Closed

Make default string size configurable in the model, use in cpp and dictionary codegen#909
jwest115 wants to merge 4 commits intonasa:mainfrom
jwest115:default-string-size

Conversation

@jwest115
Copy link
Collaborator

  • Updated dictionary spec to require FW_FIXED_LENGTH_STRING_SIZE (PR: Update dictionary spec to require FW_FIXED_LENGTH_STRING_SIZE fprime#4667)
  • Updated cpp and dictionary codegen to use FW_FIXED_LENGTH_STRING_SIZE for string size
    • Note: Since this constant is not required by cpp codegen, I kept 80 as the default if the constant is not in the model.
  • Removed -s option from fpp-to-cpp and fpp-to-dict
  • Updated user guide

Closes #903.

@jwest115 jwest115 requested a review from bocchino January 28, 2026 20:49
@bocchino
Copy link
Collaborator

This is great work! I'm closing this PR in favor of #913, which is on a branch in the repo. This work is exposing bugs in FppTest, so we need an F Prime tandem development.

@bocchino bocchino closed this Jan 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make default string size configurable in the model

2 participants