gustavonihei commented on a change in pull request #4698:
URL: https://github.com/apache/incubator-nuttx/pull/4698#discussion_r732125121
##########
File path: sched/mqueue/mq_msgqalloc.c
##########
@@ -49,50 +49,58 @@
* attr - The mq_maxmsg attribute is used at the time that the message
* queue is created to determine the maximum number of
* messages that may be placed in the message queue.
+ * msgq - This parameter is a address of a pointer
*
* Returned Value:
- * The allocated and initialized message queue structure or NULL in the
- * event of a failure.
+ * IF the function runs successfully，will return Zero(OK),else will
+ * return a error code.
Review comment:
```suggestion
* Zero (OK) is returned on success. Otherwise, a negated errno value is
* returned to indicate the nature of the failure.
```
This phrase is widespread in NuttX codebase, how about reusing it to improve
the consistency?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
