[GitHub] [incubator-nuttx-apps] patacongo opened a new pull request #651: testing/ostest: More improvements to getopt() testing/ostest

[GitBox <gi...@apache.org>]

patacongo opened a new pull request #651:
URL: https://github.com/apache/incubator-nuttx-apps/pull/651


   ## Summary
   
   - Fix optind range checking.  optind may index through argc (to the NULL argv entry) on the last option since optind is required to always point to the next command line argument.
   - Add two more test cases were the final thing on the command line is an invalid long option.
   - Fix a check that used an older, obsoleted hard-coded value that was not updated.
   
   ## Impact
   
   This should have no impact other than to the getopt() test cases of the OS test.
   
   ## Testing
   
   Tested on the simulator using a modified NSH configuration.
   


-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #651: testing/ostest: More improvements to getopt() testing/ostest

[GitBox <gi...@apache.org>]

xiaoxiang781216 merged pull request #651:
URL: https://github.com/apache/incubator-nuttx-apps/pull/651


   


-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org