<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN" "http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<title>change log for gcc-testing (2010-05-12)</title>
</head>
<body text='#000000' bgcolor='#ffffff'>
<a name='cs1'></a>
<table border='0' cellspacing='0' cellpadding='5' width='100%' bgcolor='#eeeeee'>
<tr><td colspan='3' bgcolor='#dddddd'>
 <font color='#bb2222'><strong>jennifer</strong></font>
</td></tr>
<tr><td colspan='3' bgcolor='#dddddd'><pre>2010-05-12 Jennifer Averett <Jennifer.Averett@OARcorp.com>

        * ReportsHtml.cc: Added sort ability to size report. Resolved a bug
        that resulted in mal-formed html.
</pre></td></tr>
<tr><td width='1%'><a href="http://www.rtems.com/cgi-bin/viewcvs.cgi//gcc-testing/rtems-coverage/ChangeLog.diff?r1=text&tr1=1.241&r2=text&tr2=1.242&diff_format=h">M</a></td><td width='1%'>1.242</td><td width='100%'>rtems-coverage/ChangeLog</td></tr>
<tr><td width='1%'><a href="http://www.rtems.com/cgi-bin/viewcvs.cgi//gcc-testing/rtems-coverage/ReportsHtml.cc.diff?r1=text&tr1=1.11&r2=text&tr2=1.12&diff_format=h">M</a></td><td width='1%'>1.12</td><td width='100%'>rtems-coverage/ReportsHtml.cc</td></tr>
</table>
<pre>
<font color='#006600'>diff -u gcc-testing/rtems-coverage/ChangeLog:1.241 gcc-testing/rtems-coverage/ChangeLog:1.242
--- gcc-testing/rtems-coverage/ChangeLog:1.241  Wed May 12 14:30:54 2010
+++ gcc-testing/rtems-coverage/ChangeLog        Wed May 12 14:34:38 2010
</font><font color='#997700'>@@ -1,3 +1,8 @@
</font><font color='#000088'>+2010-05-12    Jennifer Averett <Jennifer.Averett@OARcorp.com>
+
+       * ReportsHtml.cc: Added sort ability to size report. Resolved a bug
+       that resulted in mal-formed html.
+
</font> 2010-05-12        Joel Sherrill <joel.sherrill@oarcorp.com>
 
        * covoar.cc: Process single executable file name differently.

<font color='#006600'>diff -u gcc-testing/rtems-coverage/ReportsHtml.cc:1.11 gcc-testing/rtems-coverage/ReportsHtml.cc:1.12
--- gcc-testing/rtems-coverage/ReportsHtml.cc:1.11      Wed May 12 14:22:17 2010
+++ gcc-testing/rtems-coverage/ReportsHtml.cc   Wed May 12 14:34:38 2010
</font><font color='#997700'>@@ -118,7 +118,6 @@
</font>       // Put header information into the file
       fprintf(
         aFile,
<font color='#880000'>-        "<pre class=\"code\">\n"
</font>         "<table class=\"covoar table-autosort:0 table-autofilter table-autopage:10 table-page-number:pagenum table-page-count:pages\">\n"
         "<thead>\n"
         "<tr>\n"
<font color='#997700'>@@ -149,7 +148,6 @@
</font>     // Put header information into the file
     fprintf(
       aFile,
<font color='#880000'>-      "<pre class=\"code\">\n"
</font>       "<table class=\"covoar table-autosort:0 table-autofilter table-autopage:10 table-page-number:pagenum table-page-count:pages\">\n"
       "<thead>\n"
       "<tr>\n"
<font color='#997700'>@@ -178,13 +176,14 @@
</font>     // Put header information into the file
     fprintf(
       aFile,
<font color='#880000'>-      "<table class=\"covoar-table\">\n"
-      "<tbody class=\"covoar-tbody\">\n"
-      "<tr class=\"covoar-tr covoar-tr-first\">\n"
-      "<th class=\"covoar-th\">Size</th>\n"
-      "<th class=\"covoar-th\">Symbol</th>\n"
-      "<th class=\"covoar-th\">File</th>\n"
</font><font color='#000088'>+      "<table class=\"covoar table-autosort:0 table-autofilter table-autopage:10 table-page-number:pagenum table-page-count:pages\">\n"
+      "<thead>\n"
+      "<tr>\n"
+      "<th class=\"table-sortable:numeric\" align=\"left\">Size</th>\n"
+      "<th class=\"table-sortable:default\" align=\"left\">Symbol</th>\n"
+      "<th class=\"table-sortable:default\" align=\"left\">File</th>\n"
</font>       "</tr>\n"
<font color='#000088'>+      "</thead>\n"
</font>     );
     return aFile;
   }
<font color='#997700'>@@ -412,7 +411,6 @@
</font>     );
 
     // Mark the background color different for odd and even lines.
<font color='#880000'>-    fprintf( report, "</tr>\n");
</font>     if ( ( count%2 ) == 0 )
       fprintf( report, "<tr class=\"covoar-tr covoar-tr-even\">\n");
     else
<font color='#997700'>@@ -466,7 +464,6 @@
</font> 
 
     // Mark the background color different for odd and even lines.
<font color='#880000'>-    fprintf( report, "</tr>\n");
</font>     if ( ( count%2 ) == 0 )
       fprintf( report, "<tr class=\"covoar-tr covoar-tr-even\">\n");
     else
<font color='#997700'>@@ -540,7 +537,6 @@
</font>   )
   {
     // Mark the background color different for odd and even lines.
<font color='#880000'>-    fprintf( report, "</tr>\n");
</font>     if ( ( count%2 ) == 0 )
       fprintf( report, "<tr class=\"covoar-tr covoar-tr-even\">\n");
     else
<font color='#997700'>@@ -567,7 +563,7 @@
</font>       range->lowSourceLine.c_str()
     );
 
<font color='#880000'>-
</font><font color='#000088'>+    fprintf( report, "</tr>\n");
</font> 
     return true;
   }
<font color='#997700'>@@ -580,7 +576,6 @@
</font>   )
   {
     // Mark the background color different for odd and even lines.
<font color='#880000'>-    fprintf( report, "</tr>\n");
</font>     if ( ( count%2 ) == 0 )
       fprintf( report, "<tr class=\"covoar-tr covoar-tr-even\">\n");
     else
<font color='#997700'>@@ -607,6 +602,7 @@
</font>       range->lowSourceLine.c_str()
     );
 
<font color='#000088'>+    fprintf( report, "</tr>\n");
</font>     return true;
   }
 
<font color='#997700'>@@ -680,11 +676,21 @@
</font>     FILE*  aFile
   )
   {
<font color='#880000'>-    fprintf( aFile, "</tbody>\n" );
-    fprintf( aFile, "</table>\n" );
-    fprintf( aFile, "</pre>\n" );
-    fprintf( aFile,"</body>\n");
-    fprintf( aFile,"</html>");
</font><font color='#000088'>+    fprintf(
+      aFile,
+      "<tfoot>\n"
+      "<tr>\n"
+      "<td class=\"table-page:previous\" style=\"cursor:pointer;\">&lt; &lt; Previous</td>\n"
+      "<td colspan=\"1\" style=\"text-align:center;\">Page <span id=\"pagenum\"></span>&nbsp;of <span id=\"pages\"></span></td>\n"
+      "<td class=\"table-page:next\" style=\"cursor:pointer;\">Next &gt; &gt;</td>\n"
+      "</tr>\n"
+      "</tfoot>\n"
+      "</tbody>\n"
+      "</table>\n"<span style="background-color: #FF0000"> </span>
+      "</pre>\n"<span style="background-color: #FF0000"> </span>
+      "</body>\n"
+      "</html>"
+    );
</font> 
     CloseFile( aFile );
   }
</pre>
<p> </p>
<a name='cs2'></a>
<table border='0' cellspacing='0' cellpadding='5' width='100%' bgcolor='#eeeeee'>
<tr><td colspan='3' bgcolor='#dddddd'>
 <font color='#bb2222'><strong>joel</strong></font>
</td></tr>
<tr><td colspan='3' bgcolor='#dddddd'><pre>2010-05-12 Joel Sherrill <joel.sherrill@oarcorp.com>

        * ReportsHtml.cc, ReportsHtml.h, do_coverage: Tidy up reports.
</pre></td></tr>
<tr><td width='1%'><a href="http://www.rtems.com/cgi-bin/viewcvs.cgi//gcc-testing/rtems-coverage/ChangeLog.diff?r1=text&tr1=1.239&r2=text&tr2=1.240&diff_format=h">M</a></td><td width='1%'>1.240</td><td width='100%'>rtems-coverage/ChangeLog</td></tr>
<tr><td width='1%'><a href="http://www.rtems.com/cgi-bin/viewcvs.cgi//gcc-testing/rtems-coverage/ReportsHtml.h.diff?r1=text&tr1=1.7&r2=text&tr2=1.8&diff_format=h">M</a></td><td width='1%'>1.8</td><td width='100%'>rtems-coverage/ReportsHtml.h</td></tr>
<tr><td width='1%'><a href="http://www.rtems.com/cgi-bin/viewcvs.cgi//gcc-testing/rtems-coverage/ReportsHtml.cc.diff?r1=text&tr1=1.10&r2=text&tr2=1.11&diff_format=h">M</a></td><td width='1%'>1.11</td><td width='100%'>rtems-coverage/ReportsHtml.cc</td></tr>
<tr><td width='1%'><a href="http://www.rtems.com/cgi-bin/viewcvs.cgi//gcc-testing/rtems-coverage/do_coverage.diff?r1=text&tr1=1.56&r2=text&tr2=1.57&diff_format=h">M</a></td><td width='1%'>1.57</td><td width='100%'>rtems-coverage/do_coverage</td></tr>
</table>
<pre>
<font color='#006600'>diff -u gcc-testing/rtems-coverage/ChangeLog:1.239 gcc-testing/rtems-coverage/ChangeLog:1.240
--- gcc-testing/rtems-coverage/ChangeLog:1.239  Wed May 12 14:09:18 2010
+++ gcc-testing/rtems-coverage/ChangeLog        Wed May 12 14:22:17 2010
</font><font color='#997700'>@@ -1,3 +1,7 @@
</font><font color='#000088'>+2010-05-12    Joel Sherrill <joel.sherrill@oarcorp.com>
+
+       * ReportsHtml.cc, ReportsHtml.h, do_coverage: Tidy up reports.
+
</font> 2010-05-12        Joel Sherrill <joel.sherrilL@OARcorp.com>
 
        * do_coverage: Attempt to add output directory.

<font color='#006600'>diff -u gcc-testing/rtems-coverage/ReportsHtml.h:1.7 gcc-testing/rtems-coverage/ReportsHtml.h:1.8
--- gcc-testing/rtems-coverage/ReportsHtml.h:1.7        Wed May 12 13:12:36 2010
+++ gcc-testing/rtems-coverage/ReportsHtml.h    Wed May 12 14:22:17 2010
</font><font color='#997700'>@@ -68,6 +68,10 @@
</font> 
     AnnotatedLineState_t lastState_m;
 
<font color='#000088'>+    virtual FILE* OpenAnnotatedFile(
+      const char* const fileName
+    );
+
</font>     virtual FILE* OpenBranchFile(
       const char* const fileName,
       bool              hasBranches

<font color='#006600'>diff -u gcc-testing/rtems-coverage/ReportsHtml.cc:1.10 gcc-testing/rtems-coverage/ReportsHtml.cc:1.11
--- gcc-testing/rtems-coverage/ReportsHtml.cc:1.10      Wed May 12 13:37:58 2010
+++ gcc-testing/rtems-coverage/ReportsHtml.cc   Wed May 12 14:22:17 2010
</font><font color='#997700'>@@ -82,6 +82,22 @@
</font>       "<link rel=\"stylesheet\" type=\"text/css\" href=\"covoar.css\" media=\"screen\" >\n"
       "<script type=\"text/javascript\" src=\"table.js\"></script>\n"
       "<body>\n"
<font color='#000088'>+    );
+
+    return aFile;
+  }
+
+  FILE* ReportsHtml::OpenAnnotatedFile(
+    const char* const fileName
+  )
+  {
+    FILE *aFile;
+
+    // Open the file
+    aFile = OpenFile(fileName);
+
+    fprintf(
+      aFile,
</font>       "<pre class=\"code\">\n"
     );
 
<font color='#997700'>@@ -387,10 +403,12 @@
</font>     Coverage::Explanation explanation;
 
     explanation.explanation.push_back(
<font color='#000088'>+      "<html><p>\n"
</font>       "This symbol was never referenced by an analyzed executable.  "
       "Therefore there is no size or disassembly for this symbol.  "
<font color='#880000'>-      "This could be due to symbol misspelling or lack of a test for"
</font><font color='#000088'>+      "This could be due to symbol misspelling or lack of a test for "
</font>       "this symbol."
<font color='#000088'>+      "</p></html>\n"
</font>     );
 
     // Mark the background color different for odd and even lines.

<font color='#006600'>diff -u gcc-testing/rtems-coverage/do_coverage:1.56 gcc-testing/rtems-coverage/do_coverage:1.57
--- gcc-testing/rtems-coverage/do_coverage:1.56 Wed May 12 14:09:18 2010
+++ gcc-testing/rtems-coverage/do_coverage      Wed May 12 14:22:17 2010
</font><font color='#997700'>@@ -625,7 +625,7 @@
</font>       results_dir=${BSP}${EXT}-${day}-${hour}${minute}
     else
       results_dir=${outputDir}
<font color='#880000'>-    endif
</font><font color='#000088'>+    fi
</font> 
     generate_symbols >${BASEDIR}/${BSP}-tests/rtems.syms
     syms=`cat ${BASEDIR}/${BSP}-tests/rtems.syms | wc -l`
</pre>
<p> </p>
<a name='cs3'></a>
<table border='0' cellspacing='0' cellpadding='5' width='100%' bgcolor='#eeeeee'>
<tr><td colspan='3' bgcolor='#dddddd'>
 <font color='#bb2222'><strong>joel</strong></font>
</td></tr>
<tr><td colspan='3' bgcolor='#dddddd'><pre>2010-05-12 Joel Sherrill <joel.sherrill@oarcorp.com>

        * covoar.cc: Process single executable file name differently.
</pre></td></tr>
<tr><td width='1%'><a href="http://www.rtems.com/cgi-bin/viewcvs.cgi//gcc-testing/rtems-coverage/ChangeLog.diff?r1=text&tr1=1.240&r2=text&tr2=1.241&diff_format=h">M</a></td><td width='1%'>1.241</td><td width='100%'>rtems-coverage/ChangeLog</td></tr>
<tr><td width='1%'><a href="http://www.rtems.com/cgi-bin/viewcvs.cgi//gcc-testing/rtems-coverage/covoar.cc.diff?r1=text&tr1=1.21&r2=text&tr2=1.22&diff_format=h">M</a></td><td width='1%'>1.22</td><td width='100%'>rtems-coverage/covoar.cc</td></tr>
</table>
<pre>
<font color='#006600'>diff -u gcc-testing/rtems-coverage/ChangeLog:1.240 gcc-testing/rtems-coverage/ChangeLog:1.241
--- gcc-testing/rtems-coverage/ChangeLog:1.240  Wed May 12 14:22:17 2010
+++ gcc-testing/rtems-coverage/ChangeLog        Wed May 12 14:30:54 2010
</font><font color='#997700'>@@ -1,5 +1,9 @@
</font> 2010-05-12        Joel Sherrill <joel.sherrill@oarcorp.com>
 
<font color='#000088'>+   * covoar.cc: Process single executable file name differently.
+
+2010-05-12     Joel Sherrill <joel.sherrill@oarcorp.com>
+
</font>   * ReportsHtml.cc, ReportsHtml.h, do_coverage: Tidy up reports.
 
 2010-05-12     Joel Sherrill <joel.sherrilL@OARcorp.com>

<font color='#006600'>diff -u gcc-testing/rtems-coverage/covoar.cc:1.21 gcc-testing/rtems-coverage/covoar.cc:1.22
--- gcc-testing/rtems-coverage/covoar.cc:1.21   Wed May 12 08:33:30 2010
+++ gcc-testing/rtems-coverage/covoar.cc        Wed May 12 14:30:55 2010
</font><font color='#997700'>@@ -40,7 +40,6 @@
</font> std::list<Coverage::ExecutableInfo*> executablesToAnalyze;
 const char*                          explanations = NULL;
 char*                                progname;
<font color='#880000'>-bool                                 singleExecutable = false;
</font> const char*                          symbolsFile = NULL;
 const char*                          target = NULL;
 const char*                          format = NULL;
<font color='#997700'>@@ -145,6 +144,7 @@
</font>   Coverage::ExecutableInfo*                      executableInfo = NULL;
   int                                            i;
   int                                            opt;
<font color='#000088'>+  const char*                                    singleExecutable = NULL;
</font> 
   CoverageConfiguration = new Configuration::FileReader(Options);
   
<font color='#997700'>@@ -155,20 +155,10 @@
</font> 
   while ((opt = getopt(argc, argv, "1:e:c:E:f:s:T:vC:O:")) != -1) {
     switch (opt) {
<font color='#880000'>-      case '1':
-        singleExecutable = true;
-        executableInfo = new Coverage::ExecutableInfo( optarg );
-        executablesToAnalyze.push_back( executableInfo );
-        break;
-      case 'e':
-        executableExtension = optarg;
-        break;
-      case 'c':
-         coverageFileExtension = optarg;
-         break;
-      case 'C':
-         CoverageConfiguration->processFile( optarg );
-         break;
</font><font color='#000088'>+      case 'C': CoverageConfiguration->processFile( optarg ); break;
+      case '1': singleExecutable = optarg;       break;
+      case 'e': executableExtension = optarg;    break;
+      case 'c': coverageFileExtension = optarg;  break;
</font>       case 'E': explanations          = optarg;  break;
       case 's': symbolsFile           = optarg;  break;
       case 'T': target                = optarg;  break;
<font color='#997700'>@@ -181,11 +171,16 @@
</font>     }
   }
 
<font color='#000088'>+  // Do not trust any arguments until after this point.
</font>   check_configuration();
 
<font color='#000088'>+  // XXX We need to verify that all of the needed arguments are non-NULL.
+
</font>   // If a single executable was specified, process the remaining
   // arguments as coverage file names.
   if (singleExecutable) {
<font color='#000088'>+    executableInfo = new Coverage::ExecutableInfo( singleExecutable );
+    executablesToAnalyze.push_back( executableInfo );
</font>     for (i=optind; i < argc; i++) {
       coverageFileName = argv[i];
       coverageFileNames.push_back( coverageFileName );
</pre>
<p> </p>

<p>--<br />
<small>Generated by <a href="http://www.codewiz.org/projects/index.html#loginfo">Deluxe Loginfo</a> 2.122 by Bernardo Innocenti <bernie@develer.com></small></p>
</body>
</html>