Document the API as “C++”, not “C”#89
Merged
Lightsaver7 merged 1 commit intoRedPitaya:masterfrom Feb 12, 2026
Merged
Conversation
Many files still refer to the C language or API when they actually mean C++. Fix them.
Collaborator
|
Oh, I see what you mean. Yes, I missed those during the update. I will merge this and check if there are any other places where this applies. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The Red Pitaya C API has been replaced by a C++ API. The February 2026 update to the documentation does mention this fact. However, there are still lots of pages that incorrectly mention a “C API”.
This pull request replaces “C” by “C++” in many places where this is relevant. Note however that:
it does not touch the many mentions of “C” within
appsFeatures/examples, as I did not test these examples with a C++ compilerit does not rename the files
C_and_Python_API.{rst,inc}Fixes: #87.
For the record, mentions of “C” can be found with the command below, but beware of the many false positives: