Skip to content
New issue

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

Fix the format of doxygen comments of output arguments #917

Conversation

LaszloLango
Copy link
Contributor

JerryScript-DCO-1.0-Signed-off-by: László Langó [email protected]

@LaszloLango LaszloLango added the style Related to coding style label Feb 24, 2016
@@ -925,7 +925,7 @@ ecma_date_set_internal_property (ecma_value_t this_arg, /**< this argument */
* Insert leading zeros to a string of a number if needed.
*/
void
ecma_date_insert_leading_zeros (ecma_string_t **str_p, /**< input/output string */
ecma_date_insert_leading_zeros (ecma_string_t **str_p, /**< [in, out] ecma string */
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no space between in, out

@LaszloLango LaszloLango force-pushed the doxygen-fix-output-comments branch from 104a849 to 0af43f8 Compare February 25, 2016 07:13
@LaszloLango
Copy link
Contributor Author

@akiss77, fixed.

@akosthekiss
Copy link
Member

LGTM (FWIW)

@LaszloLango LaszloLango assigned zherczeg and unassigned akosthekiss Feb 25, 2016
@zherczeg
Copy link
Member

LGTM

JerryScript-DCO-1.0-Signed-off-by: László Langó [email protected]
@LaszloLango LaszloLango force-pushed the doxygen-fix-output-comments branch from 0af43f8 to e14d0b8 Compare March 1, 2016 11:38
@LaszloLango LaszloLango merged commit e14d0b8 into jerryscript-project:master Mar 1, 2016
@LaszloLango LaszloLango deleted the doxygen-fix-output-comments branch March 2, 2016 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
style Related to coding style
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants