On Sat, May 12, 2018 at 2:42 PM, thelunatic <vijaykumar9...@gmail.com>
wrote:
> ---
>
Every patch needs an explanation/log message. Look back through
"git log" for examples.
> tester/covoar/covoar.cc | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/tester/covoar/covoar.cc b/tester/covoar/covoar.cc
> index 5bed98f..95dc990 100644
> --- a/tester/covoar/covoar.cc
> +++ b/tester/covoar/covoar.cc
> @@ -185,7 +185,7 @@ int main(
> Executables executablesToAnalyze;
> Coverage::ExecutableInfo* executableInfo = NULL;
> std::string executableExtension = "exe";
> - std::string coverageExtension = "cov";
> + std::string coverageExtension = ".cov";
>
Why did this need a "." but the "exe" one didn't?
> Coverage::CoverageFormats_t coverageFormat =
> Coverage::COVERAGE_FORMAT_QEMU;
> Coverage::CoverageReaderBase* coverageReader = NULL;
> char* executable = NULL;
> @@ -317,11 +317,12 @@ int main(
> std::cerr << "warning: Unable to read executable: " << argv[i] <<
> std::endl;
> } else {
> coverageFileName = argv[i];
> - coverageFileName.replace(
> + coverageFileName.append(coverageExtension);
> + /* coverageFileName.replace(
> coverageFileName.length() - executableExtension.size(),
> executableExtension.size(),
> coverageExtension
> - );
> + ); */
>
If you are replacing this call, then just delete it -- don't comment it out.
> if (!FileIsReadable( coverageFileName.c_str() )) {
> std::cerr << "warning: Unable to read coverage file: " <<
> coverageFileName
> --
> 2.14.3
>
>
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel
