Feature #2521 sonarqube code smell

src/basic/enum_to_string/code.cc

@@ -10,9 +10,6 @@
 ////////////////////////////////////////////////////////////////////////
 
 
-using namespace std;
-
-
 #include <iostream>
 #include <fstream>
 #include <unistd.h>
@@ -29,6 +26,9 @@ using namespace std;
 #include "vx_cal.h"
 
 
+using namespace std;
+
+
 ////////////////////////////////////////////////////////////////////////
 
 

src/basic/enum_to_string/enum_to_string.cc

@@ -16,8 +16,6 @@ static const int debug = 0;
 ////////////////////////////////////////////////////////////////////////
 
 
-using namespace std;
-
 #include <iostream>
 #include <unistd.h>
 
@@ -35,6 +33,9 @@ using namespace std;
 #include "enum_to_string.h"
 
 
+using namespace std;
+
+
 ////////////////////////////////////////////////////////////////////////
 
 
@@ -157,7 +158,7 @@ if ( parse_status != 0 )  {
 
 }
 
-return ( 0 );
+return 0;
 
 }
 
@@ -171,7 +172,8 @@ void parse_command_line(int &argc, char **argv)
 
 if ( argc == 1 )  { usage();  exit ( 1 ); }
 
-int j, found;
+int j;
+int found;
 
 do {
 
@@ -217,9 +219,7 @@ void shift_down(int &argc, char **argv, int pos, int shift)
 
 {
 
-int j;
-
-for (j=pos; j<(argc - shift); ++j)  argv[j] = argv[j + shift];
+for (int j=pos; j<(argc - shift); ++j)  argv[j] = argv[j + shift];
 
 argc -= shift;
 
@@ -396,7 +396,7 @@ int yywrap()
 
 {
 
-return ( 1 );
+return 1;
 
 }
 

src/basic/enum_to_string/info.h

@@ -25,7 +25,7 @@ static const int enuminfo_alloc_increment = 100;
 
 class EnumInfo {
 
-      friend ostream & operator<<(ostream &, const EnumInfo &);
+      friend std::ostream & operator<<(std::ostream &, const EnumInfo &);
 
    private:
 
@@ -106,7 +106,7 @@ inline const char * EnumInfo::header() const { return ( Header ); }
 ////////////////////////////////////////////////////////////////////////
 
 
-extern ostream & operator<<(ostream &, const EnumInfo &);
+extern std::ostream & operator<<(std::ostream &, const EnumInfo &);
 
 
 ////////////////////////////////////////////////////////////////////////

src/basic/enum_to_string/my_enum_scanner.cc

@@ -4,9 +4,6 @@
 ////////////////////////////////////////////////////////////////////////
 
 
-using namespace std;
-
-
 #include <iostream>
 #include <unistd.h>
 #include <stdlib.h>
@@ -22,6 +19,10 @@ using namespace std;
 
 #include "str_wrappers.h"
 
+
+using namespace std;
+
+
 ////////////////////////////////////////////////////////////////////////
 
 
@@ -155,7 +156,7 @@ while ( 1 )  {
 //  
 // cout << "\n\n" << flush;
 
-return ( t );
+return t;
 
 }
 
@@ -266,7 +267,7 @@ if ( is_id() )  { if ( do_id() )  return ( token(ID) ); }
 
 
 
-return ( skip );
+return skip;
 
 }
 
@@ -330,7 +331,7 @@ if ( k == char_class_digit )  return ( true );
 if ( k == char_class_alpha )  return ( true );
 
 
-return ( false );
+return false;
 
 }
 
@@ -378,9 +379,9 @@ bool is_id()
 
 if ( char_class[lexeme[0]] != char_class_alpha )  return ( false );
 
-int j, k;
+int k;
 
-for (j=0; j<max_lexeme_size; ++j)  {
+for (int j=0; j<max_lexeme_size; ++j)  {
 
    if ( lexeme[j] == 0 )  break;
 
@@ -438,13 +439,13 @@ if ( enum_mode || last_was_enum || last_was_class )  {
 
    m_strncpy(yylval.name, yytext, sizeof(yylval.name), method_name);
 
-   return ( 1 );
+   return 1;
 
 }
 
 
 
-return ( 0 );
+return 0;
 
 }
 
@@ -456,13 +457,13 @@ int do_int()
 
 {
 
-if ( !enum_mode )  return ( 0 );
+if ( !enum_mode )  return 0;
 
 yylval.ival = atoi(yytext);
 
 
 
-return ( 1 );
+return 1;
 
 }
 
@@ -478,14 +479,14 @@ int do_left_curly()
 
 ++column;
 
-if ( !enum_mode )  return ( 0 );
+if ( !enum_mode )  return 0;
 
 yylval.ival = bracket_level;
 
 lexeme[0] = '{';
 
 
-return ( 1 );
+return 1;
 
 }
 
@@ -503,14 +504,14 @@ int do_right_curly()
 
 ss.clear_to_level(bracket_level);
 
-if ( !enum_mode )  return ( 0 );
+if ( !enum_mode )  return 0;
 
 yylval.ival = bracket_level + 1;
 
 lexeme[0] = '}';
 
 
-return ( 1 );
+return 1;
 
 }
 
@@ -528,13 +529,13 @@ if ( enum_mode )  {
 
    lexeme[0] = '}';
 
-   return ( 1 );
+   return 1;
 
 }
 
 
 
-return ( 0 );
+return 0;
 
 }
 
@@ -552,12 +553,12 @@ if ( enum_mode )  {
 
    lexeme[0] = '=';
 
-   return ( 1 );
+   return 1;
 
 }
 
 
-return ( 0 );
+return 0;
 
 }
 
@@ -575,12 +576,12 @@ if ( enum_mode )  {
 
    lexeme[0] = ',';
 
-   return ( 1 );
+   return 1;
 
 }
 
 
-return ( 0 );
+return 0;
 
 }
 
@@ -669,7 +670,7 @@ if ( c == '\n' )  {
 }
 
 
-return ( c );
+return c;
 
 }
 
@@ -691,7 +692,7 @@ if ( t == CLASS )  last_was_class = 1;
 if ( t == ENUM  )  last_was_enum  = 1;
 
 
-return ( t );
+return t;
 
 }
 

src/basic/enum_to_string/scope.cc

@@ -10,9 +10,6 @@
 ////////////////////////////////////////////////////////////////////////
 
 
-using namespace std;
-
-
 #include <iostream>
 #include <unistd.h>
 #include <stdlib.h>
@@ -23,6 +20,8 @@ using namespace std;
 #include "str_wrappers.h"
 
 
+using namespace std;
+
 
 ////////////////////////////////////////////////////////////////////////
 
@@ -83,7 +82,7 @@ if ( this == &e )  return ( *this );
 
 assign(e);
 
-return ( *this );
+return *this;
 
 }
 
@@ -239,7 +238,7 @@ if ( this == &ss )  return ( *this );
 
 assign(ss);
 
-return ( *this );
+return *this;
 
 }
 
@@ -313,7 +312,7 @@ if ( (pos < 0) || (pos >= N) )  {
 }
 
 
-return ( s[pos] );
+return s[pos];
 
 }
 
@@ -437,7 +436,7 @@ s << "   level = "   << (sse.level()) << "\n";
 
 s.flush();
 
-return ( s );
+return s;
 
 }
 
@@ -474,7 +473,7 @@ for (j=0; j<n; ++j)  {
 }
 
 
-return ( s );
+return s;
 
 }
 

src/basic/enum_to_string/scope.h

@@ -65,7 +65,7 @@ inline const char * ScopeStackElement::name() const { return ( Name ); }
 ////////////////////////////////////////////////////////////////////////
 
 
-extern ostream & operator<<(ostream &, const ScopeStackElement &);
+extern std::ostream & operator<<(std::ostream &, const ScopeStackElement &);
 
 
 ////////////////////////////////////////////////////////////////////////
@@ -119,7 +119,7 @@ inline int ScopeStack::n_elements() const { return ( N ); }
 ////////////////////////////////////////////////////////////////////////
 
 
-extern ostream & operator<<(ostream &, const ScopeStack &);
+extern std::ostream & operator<<(std::ostream &, const ScopeStack &);
 
 
 ////////////////////////////////////////////////////////////////////////