[GitHub] [incubator-nuttx] patacongo opened a new issue #387: nxstyle: Suppress checking in strings/comments.

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

patacongo opened a new issue #387: nxstyle:  Suppress checking in strings/comments.
URL: https://github.com/apache/incubator-nuttx/issues/387
 
 
   nxstyle does not detect when it is within a string and will report crazy things because it is parsing the strng content.  For example, if the string contains "Hello", nxstyle would probleam complain about mixed case identifiers, if the strong contains "https://", it would probably complete about  C++ style comments.  If the string contains "yes/no" it would probably complain about missing spaces before and after an operator.
   
   Below is a even more obsure false alarm example:
   
   junk.c:
   
       /****************************************************************************
        * Private Data
        ****************************************************************************/
       
       static const char g_blabber[] = "Yadda, yadda, yadda, yadda";
       
       /****************************************************************************
       * Public Functions
       ****************************************************************************/
   
   then you see:
   
       $ tools/nxstyle.exe junk.c
       junk.c:5:39: error: Multiple data definitions
   
   most of this also applies to comments.  You will see the same kinds of errors for similer sub-strings within a comment.  nxstyle does detect that it is within a comment, however, it does not use that information to suppress certain error reports.
   

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


With regards,
Apache Git Services