Changeset 149144f in rtems-tools


Ignore:
Timestamp:
Dec 5, 2017, 1:52:21 PM (18 months ago)
Author:
Joel Sherrill <joel@…>
Branches:
master
Children:
b516a58
Parents:
c432d89
git-author:
Joel Sherrill <joel@…> (12/05/17 13:52:21)
git-committer:
Joel Sherrill <joel@…> (12/05/17 14:01:49)
Message:

tester/covoar/covoar.cc: Clean up formatting (tabs, braces, indentation, etc)

File:
1 edited

Legend:

Unmodified
Added
Removed
  • tester/covoar/covoar.cc

    rc432d89 r149144f  
    4545char*                                progname;
    4646const char*                          symbolsFile = NULL;
    47 const char*                          gcnosFileName = NULL;
    48 char                                 gcnoFileName[FILE_NAME_LENGTH];
    49 char                                 gcdaFileName[FILE_NAME_LENGTH];
    50 char                                 gcovBashCommand[256];
     47const char*                          gcnosFileName = NULL;
     48char                                 gcnoFileName[FILE_NAME_LENGTH];
     49char                                 gcdaFileName[FILE_NAME_LENGTH];
     50char                                 gcovBashCommand[256];
    5151const char*                          target = NULL;
    5252const char*                          format = NULL;
    53 FILE*                                gcnosFile = NULL;
     53FILE*                                gcnosFile = NULL;
    5454Gcov::GcovData*          gcovFile;
    5555
     
    167167  try
    168168  {
    169    /*
    170         * Validate inputs.
    171         */
    172 
    173    /*
    174         * Target name must be set.
    175         */
    176         if ( !target ) {
    177           option = "target -T";
    178           throw option;
    179         }
    180 
    181    /*
    182         * Validate simulator format.
    183         */
    184         if ( !format ) {
    185           option = "format -f";
    186           throw option;
    187         }
    188 
    189    /*
    190         * Has path to explanations.txt been specified.
    191         */
    192         if ( !explanations ) {
    193           option = "explanations -E";
    194           throw option;
    195         }
    196 
    197    /*
    198         * Has coverage file extension been specified.
    199         */
    200         if ( !coverageFileExtension ) {
    201           option = "coverage extension -c";
    202           throw option;
    203         }
    204 
    205    /*
    206         * Has executable extension been specified.
    207         */
    208         if ( !executableExtension ) {
    209           option = "executable extension -e";
    210           throw option;
    211         }
    212 
    213    /*
    214         * Check for project name.
    215         */
    216         if ( !projectName ) {
    217           option = "project name -p";
    218           throw option;
    219         }
     169    /*
     170     * Validate inputs.
     171     */
     172
     173    /*
     174     * Target name must be set.
     175     */
     176    if ( !target ) {
     177      option = "target -T";
     178      option;
     179    }
     180
     181    /*
     182     * Validate simulator format.
     183     */
     184    if ( !format ) {
     185      option = "format -f";
     186      throw option;
     187    }
     188
     189    /*
     190     * Has path to explanations.txt been specified.
     191     */
     192    if ( !explanations ) {
     193      option = "explanations -E";
     194      throw option;
     195    }
     196
     197    /*
     198     * Has coverage file extension been specified.
     199     */
     200    if ( !coverageFileExtension ) {
     201      option = "coverage extension -c";
     202      throw option;
     203    }
     204
     205    /*
     206     * Has executable extension been specified.
     207     */
     208    if ( !executableExtension ) {
     209      option = "executable extension -e";
     210      throw option;
     211    }
     212
     213    /*
     214     * Check for project name.
     215     */
     216    if ( !projectName ) {
     217      option = "project name -p";
     218      throw option;
     219    }
    220220  }
    221221  catch( std::string option )
    222222  {
    223         std::cout << "error missing option: " + option << std::endl;
    224         usage();
    225         throw;
     223    std::cout << "error missing option: " + option << std::endl;
     224    usage();
     225    throw;
    226226  }
    227227
     
    237237        singleExecutable
    238238      );
    239     }
    240 
    241     else {
     239    } else {
    242240
    243241      for (i=optind; i < argc; i++) {
    244 
    245242        // Ensure that the coverage file is readable.
    246243        if (!FileIsReadable( argv[i] )) {
     
    250247            argv[i]
    251248          );
     249        } else {
     250          coverageFileNames.push_back( argv[i] );
    252251        }
    253 
    254         else
    255           coverageFileNames.push_back( argv[i] );
    256252      }
    257253
     
    259255      // executable information.
    260256      if (!coverageFileNames.empty()) {
    261         if (dynamicLibrary)
     257        if (dynamicLibrary) {
    262258          executableInfo = new Coverage::ExecutableInfo(
    263259            singleExecutable, dynamicLibrary
    264260          );
    265         else
     261        } else {
    266262          executableInfo = new Coverage::ExecutableInfo( singleExecutable );
     263        }
    267264
    268265        executablesToAnalyze.push_back( executableInfo );
     
    283280          argv[i]
    284281        );
    285       }
    286 
    287       else {
     282      } else {
    288283        coverageFileName = argv[i];
    289284        coverageFileName.replace(
     
    299294            coverageFileName.c_str()
    300295          );
    301         }
    302 
    303         else {
     296        } else {
    304297          executableInfo = new Coverage::ExecutableInfo( argv[i] );
    305298          executablesToAnalyze.push_back( executableInfo );
     
    319312
    320313  if (Verbose) {
    321     if (singleExecutable)
     314    if (singleExecutable) {
    322315      fprintf(
    323316        stderr,
    324317        "Processing a single executable and multiple coverage files\n"
    325318      );
    326     else
     319    } else {
    327320      fprintf(
    328321        stderr,
    329322        "Processing multiple executable/coverage file pairs\n"
    330323      );
     324    }
    331325    fprintf( stderr, "Coverage Format : %s\n", format );
    332326    fprintf( stderr, "Target          : %s\n", PrintableString(target) );
     
    336330    eitr = executablesToAnalyze.begin();
    337331    for (citr = coverageFileNames.begin();
    338         citr != coverageFileNames.end();
    339         citr++) {
    340 
    341         fprintf(
    342           stderr,
    343           "Coverage file %s for executable %s\n",
    344           (*citr).c_str(),
    345           ((*eitr)->getFileName()).c_str()
    346         );
    347 
    348         if (!singleExecutable)
    349           eitr++;
     332        citr != coverageFileNames.end();
     333        citr++) {
     334
     335      fprintf(
     336        stderr,
     337        "Coverage file %s for executable %s\n",
     338        (*citr).c_str(),
     339        ((*eitr)->getFileName()).c_str()
     340      );
     341
     342      if (!singleExecutable)
     343        eitr++;
    350344    }
    351345#endif
     
    362356  SymbolsToAnalyze = new Coverage::DesiredSymbols();
    363357  SymbolsToAnalyze->load( symbolsFile );
    364   if (Verbose)
     358  if (Verbose) {
    365359    fprintf(
    366360      stderr,
     
    368362      (unsigned int) SymbolsToAnalyze->set.size()
    369363    );
     364  }
    370365
    371366  // Create explanations.
     
    389384       eitr++) {
    390385
    391     if (Verbose)
     386    if (Verbose) {
    392387      fprintf(
    393388        stderr,
     
    395390        ((*eitr)->getFileName()).c_str()
    396391      );
     392    }
    397393
    398394    // If a dynamic library was specified, determine the load address.
    399     if (dynamicLibrary)
     395    if (dynamicLibrary) {
    400396      (*eitr)->setLoadAddress(
    401397        objdumpProcessor->determineLoadAddress( *eitr )
    402398      );
     399    }
    403400
    404401    // Load the objdump for the symbols in this executable.
     
    416413       citr++) {
    417414
    418     if (Verbose)
     415    if (Verbose) {
    419416      fprintf(
    420417        stderr,
     
    423420        ((*eitr)->getFileName()).c_str()
    424421      );
     422    }
    425423
    426424    // Process its coverage file.
     
    433431    //(*eitr)->dumpExecutableInfo();
    434432
    435     if (!singleExecutable)
     433    if (!singleExecutable) {
    436434      eitr++;
     435    }
    437436  }
    438437
    439438  // Do necessary preprocessing of uncovered ranges and branches
    440   if (Verbose)
     439  if (Verbose) {
    441440    fprintf( stderr, "Preprocess uncovered ranges and branches\n" );
     441  }
    442442  SymbolsToAnalyze->preprocess();
    443443
     
    445445  // Generate Gcov reports
    446446  //
    447   if (Verbose)
     447  if (Verbose) {
    448448    fprintf( stderr, "Generating Gcov reports...\n");
     449  }
    449450  gcnosFile = fopen ( gcnosFileName , "r" );
    450451
     
    457458      strcpy( gcnoFileName, inputBuffer );
    458459
    459       if ( Verbose )
    460         fprintf( stderr, "Processing file: %s\n", gcnoFileName );
     460      if ( Verbose ) {
     461        fprintf( stderr, "Processing file: %s\n", gcnoFileName );
     462      }
    461463
    462464      if ( gcovFile->readGcnoFile( gcnoFileName ) ) {
     
    470472      delete gcovFile;
    471473    }
    472   fclose( gcnosFile );
     474    fclose( gcnosFile );
    473475  }
    474476
    475477  // Determine the uncovered ranges and branches.
    476   if (Verbose)
     478  if (Verbose) {
    477479    fprintf( stderr, "Computing uncovered ranges and branches\n" );
     480  }
    478481  SymbolsToAnalyze->computeUncovered();
    479482
    480483  // Calculate remainder of statistics.
    481   if (Verbose)
     484  if (Verbose) {
    482485    fprintf( stderr, "Calculate statistics\n" );
     486  }
    483487  SymbolsToAnalyze->calculateStatistics();
    484488
    485489  // Look up the source lines for any uncovered ranges and branches.
    486   if (Verbose)
     490  if (Verbose) {
    487491    fprintf(
    488492      stderr, "Looking up source lines for uncovered ranges and branches\n"
    489493    );
     494  }
    490495  SymbolsToAnalyze->findSourceForUncovered();
    491496
     
    493498  // Report the coverage data.
    494499  //
    495   if (Verbose)
     500  if (Verbose) {
    496501    fprintf(
    497502      stderr, "Generate Reports\n"
    498503    );
     504  }
    499505  Coverage::GenerateReports();
    500506
     
    507513    notFound += "ExplanationsNotFound.txt";
    508514
    509     if (Verbose)
     515    if (Verbose) {
    510516      fprintf( stderr, "Writing Not Found Report (%s)\n", notFound.c_str() );
     517    }
    511518    AllExplanations->writeNotFound( notFound.c_str() );
    512519  }
     
    514521  //Leave tempfiles around if debug flag (-d) is enabled.
    515522  if ( debug ) {
    516         objdumpFile.override( "objdump_file" );
    517         objdumpFile.keep();
    518         err.override( "objdump_exec_log" );
    519         err.keep();
     523    objdumpFile.override( "objdump_file" );
     524    objdumpFile.keep();
     525    err.override( "objdump_exec_log" );
     526    err.keep();
    520527  }
    521528  return 0;
Note: See TracChangeset for help on using the changeset viewer.