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

MYNEWT-767 Update OS Guide for New Shell #217

Merged
merged 3 commits into from
Jul 7, 2017

Conversation

cwanda
Copy link
Contributor

@cwanda cwanda commented Jul 5, 2017

No description provided.

@cwanda cwanda changed the title MYNEWT-767 Documentation for new shell. MYNEWT-767 Update OS Guide for New Shell Jul 5, 2017
typedef int (*shell_cmd_func_t)(int argc, char *argv[]);
```

The `argc` parameter specifies the number of command line arguments and the `argv` parameter is an array of character pointers to the command arguments. The `SHELL_CMD_ARGC_MAX` syscfg setting specifies the maximum number of command line arguments that any shell command can have. This value must be increased if a shell command requires more than `SHELL_CMD_MAX_ARGC_MAX` number of command line arguments.

Choose a reason for hiding this comment

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

SHELL_CMD_MAX_ARGC_MAX I believe this is a typo

Copy link

@michal-narajowski michal-narajowski left a comment

Choose a reason for hiding this comment

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

Looks good to me

@ccollins476ad ccollins476ad merged commit 6a89da1 into apache:develop Jul 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants