Codestriker: collaborative code reviewer

  User: sits    
  Date: 06/07/02 22:06:48

  Modified:    lib/Codestriker/Repository ClearCaseDynamic.pm
                        ClearCaseSnapshot.pm
  Log:
  Missing brackets which means the error processing doesn't work correctly.
  
  
  
  Index: ClearCaseDynamic.pm
  ===================================================================
  RCS file: /cvsroot/codestriker/codestriker/lib/Codestriker/Repository/ClearCaseDynamic.pm,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- ClearCaseDynamic.pm	21 Jun 2006 04:58:49 -0000	1.2
  +++ ClearCaseDynamic.pm	3 Jul 2006 05:06:48 -0000	1.3
  @@ -67,7 +67,7 @@
   	}
   
   	# Load the file directly into the given array.
  -	open CONTENTFILE, "$full_element_name"
  +	open (CONTENTFILE, "$full_element_name")
   	    || die "Couldn't open file: $full_element_name: $!";
   	for (my $i = 1; <CONTENTFILE>; $i++) {
   	    chop;
  
  
  
  
  
  Index: ClearCaseSnapshot.pm
  ===================================================================
  RCS file: /cvsroot/codestriker/codestriker/lib/Codestriker/Repository/ClearCaseSnapshot.pm,v
  retrieving revision 1.5
  retrieving revision 1.6
  diff -u -r1.5 -r1.6
  --- ClearCaseSnapshot.pm	10 Jun 2006 02:58:34 -0000	1.5
  +++ ClearCaseSnapshot.pm	3 Jul 2006 05:06:48 -0000	1.6
  @@ -63,14 +63,14 @@
               # If there was an error, the message will be in the error file.
               # Read in that file and store it in the "$error_msg" variable
               # so that we can return it to the caller.
  -            open ERRORFILE, "<$errorfile"
  +            open (ERRORFILE, "<$errorfile")
                   || die "ClearTool returned an error, but Codestriker couldn't read from the error file.";
               my (@errorlines) = <ERRORFILE>;
               $error_msg = "Error from ClearTool: " . join(" ", @errorlines);
               close ERRORFILE;
           } else {
               # Operation was successful.  Load the file into the given array.
  -            open CONTENTFILE, "<$tempfile"
  +            open (CONTENTFILE, "<$tempfile")
                   || die "ClearTool execution succeeded, but Codestriker couldn't read from the output file.";
               for (my $i = 1; <CONTENTFILE>; $i++) {
   		$_ = Codestriker::decode_topic_text($_);