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

doxygen misc. warning #280

Closed
avan989 opened this issue Oct 30, 2019 · 0 comments · Fixed by #282 or #292
Closed

doxygen misc. warning #280

avan989 opened this issue Oct 30, 2019 · 0 comments · Fixed by #282 or #292
Milestone

Comments

@avan989
Copy link
Contributor

avan989 commented Oct 30, 2019

Describe the bug
osal/src/os/inc/osapi-os-core.h:233: warning: The following parameters of OS_ConvertToArrayIndex(uint32 object_id, uint32 *ArrayIndex) are not documented:
parameter 'ArrayIndex'

osal/src/os/inc/osapi-os-core.h:311: warning: argument 'milliseconds' of command @param is not found in the argument list of OS_TaskDelay(uint32 millisecond)

osal/src/os/inc/osapi-os-core.h:319: warning: The following parameters of OS_TaskDelay(uint32 millisecond) are not documented:
parameter 'millisecond'

osal/src/os/inc/osapi-os-core.h:332: warning: The following parameters of OS_TaskSetPriority(uint32 task_id, uint32 new_priority) are not documented:
parameter 'new_priority'

osal/src/os/inc/osapi-os-filesys.h:247: warning: The following parameters of OS_open(const char *path, int32 access, uint32 mode) are not documented:
parameter 'mode'

To Reproduce
Steps to reproduce the behavior:

  1. make usersguide
  2. See error

System observed on:

  • Hardware
  • Ubuntu 18.04
  • doxygen 1.8.13, rc-6.7.0

Reporter Info
Anh Van, NASA Goddard

@skliper skliper added this to the 5.1.0 milestone Nov 1, 2019
skliper pushed a commit that referenced this issue Nov 15, 2019
skliper pushed a commit that referenced this issue Nov 15, 2019
skliper added a commit that referenced this issue Nov 15, 2019
Fixes #286, fixes #280
Code reviewed and approved at 20191106 CCB
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants