Skip to content

Implied uses for strings#928

Merged
bocchino merged 43 commits intomainfrom
issue-927-implied-string-deps
Feb 26, 2026
Merged

Implied uses for strings#928
bocchino merged 43 commits intomainfrom
issue-927-implied-string-deps

Conversation

@bocchino
Copy link
Copy Markdown
Collaborator

@bocchino bocchino commented Feb 25, 2026 •
edited
Loading

  • Refactor how implied uses are handled, to improve code quality, extensibility, and error reporting
  • Add implied uses for strings
  • Update spec, unit tests, and User's Guide to match
  • Add unit tests

Closes #927.

jwest115 and others added 30 commits February 5, 2026 11:17
@jwest115
Require default string size to be defined when using string types
4f40ebd
@jwest115
Update spec
898b2a3
@jwest115
Merge remote-tracking branch 'upstream/refactor-framework-defs' into …
1a2d6c5 
…string-size
@jwest115
Merge remote-tracking branch 'upstream/refactor-implied-uses' into st…
0912101 
…ring-size
@jwest115
Require FwSizeStoreType and default string size (if not specified) wh…
e04c165 
…en using strings, update tests, regen ref files
@jwest115
Merge branch 'main' into string-size
d3c1ab9
@jwest115
Update error message
446408c
@jwest115
Update spec and user guide
d0e48ff
@jwest115
Add missing test
a91914d
@jwest115
Cleanup
1ed2cf9
@bocchino
Merge remote-tracking branch 'nasa/main' into string-size
1dab0b9
@bocchino
Fix bug in error reporting for cycle detection
a02e370
@bocchino
Refactor CheckUses; revise error reporting
1dbe5a8
@bocchino
Refactor ConstructImpliedUseMap
ff83379
@bocchino
Refactor ConstructImpliedUseMap
413514f
@bocchino
Revise fpp-depend tests
9371e95
@bocchino
Revise tests for fpp-locate-uses
358a784
@bocchino
Revise tests for fpp-check
3a87685
@bocchino
Revise CheckUses
ce12aed
@bocchino
Revise CheckUses
1a828e9 
Revise tests to match
@bocchino
Revise CheckUses; add tests
8dab675
@bocchino
Refactor CheckUses
c729649
@bocchino
Revise fpp-to-cpp tests
07e0ff1
@bocchino
Revise fpp-to-cpp state machine tests
bfc2cd4
@bocchino
Revise test scripts and config for fpp-to-cpp
771087b
@bocchino
Revise test script
b8c0099
@bocchino
Revise fpp-check tests
1cefeb0
@bocchino
Reformat test code
2593498
@bocchino
Add comment in test input
e87d245
@bocchino
Revise spec
912881a
@bocchino bocchino requested review from Kronos3 and jwest115 February 25, 2026 19:44
@bocchino bocchino merged commit f24ac60 into main Feb 26, 2026
17 checks passed
@bocchino bocchino deleted the issue-927-implied-string-deps branch February 26, 2026 23:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jwest115 jwest115 jwest115 approved these changes

@Kronos3 Kronos3 Awaiting requested review from Kronos3

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add implied uses for string types

2 participants

@bocchino @jwest115