We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
If the function signature contains required and optional arguments, Igor Pro does allow the following four cases of comma placement:
Function Case1(string reqArg, [string optArg])
Function Case2(string reqArg[, string optArg])
Function Case3(string reqArg [string optArg])
Function Case4(string reqArg, [, string optArg])
I would allow only case 1 as good coding style and also looks cleaner.
I have seen case 2 and case 3 in many different repositories, so I wouldn't report this to WM for compatibility reasons.
The text was updated successfully, but these errors were encountered:
Code Style: Unify comma in function signatures
136dc28
See byte-physics/igor-pro-coding-conventions#34
349f242
40879cc
No branches or pull requests
If the function signature contains required and optional arguments, Igor Pro does allow the following four cases of comma placement:
I would allow only case 1 as good coding style and also looks cleaner.
Impact
I have seen case 2 and case 3 in many different repositories, so I wouldn't report this to WM for compatibility reasons.
The text was updated successfully, but these errors were encountered: