nasa / fprime

F´ - A flight software and embedded systems framework
https://fprime.jpl.nasa.gov
Apache License 2.0
10.03k stars 1.29k forks source link

Removed a_ prefix on arguments #2513

Closed JohanBertrand closed 7 months ago

JohanBertrand commented 7 months ago
Related Issue(s) https://github.com/nasa/fprime/issues/2492
Has Unit Tests (y/n) yes
Documentation Included (y/n) yes

Change Description

Removed the prefix _a previously added in https://github.com/nasa/fprime/pull/2482

Changes:

Rationale

The a_ prefix should not be used for arguments

Testing/Review Recommendations

I am not sure if there are some breaking changes for other projects

bocchino commented 7 months ago

Function implementations in the header file are fine if they are short functions or they are part of a template.