-
Notifications
You must be signed in to change notification settings - Fork 52
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add codegen support for Daqmx Read apis which have multiple out param…
…s. (#1129) * add new meesage to proto for moniker streaming call * work-in-progress: codegen changes for service.cpp generation with response on moniker out param * run codegen after resolving merge conflicts * return failure error code from moniker APIs * update proto and struct to contain request and response messages * standardize to send output in response parameter for moniker APIs * fix black formatter issues * fix build error * cleanup scripts * fix tests * script and cpp cleanup * codegen for read apis with multiple out params * added condition to call reserve and resize functions for some param types * merged latest main --------- Co-authored-by: Rohan Doshi <rohan.doshi@ni.com> Co-authored-by: Raghav Rawat <raghavrawat@ni.com>
- Loading branch information
1 parent
06ed455
commit b966222
Showing
11 changed files
with
2,108 additions
and
381 deletions.
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains 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
Oops, something went wrong.