Rename $cmd_class to $cmd_classes.

Makes code easier to read.

1 files changed, 10 insertions, 9 deletions

diff --git a/lib/server/makeprotocol.pl.in b/lib/server/makeprotocol.pl.in
index 0e6f7351..6abffc7a 100755
--- a/lib/server/makeprotocol.pl.in
+++ b/lib/server/makeprotocol.pl.in
@@ -243,7 +243,8 @@ std::auto_ptr<$message_base_class> $message_base_class\::DoCommand($replyable_ba
 }
 __E
 
-my %cmd_class;
+my %cmd_classes;
+my $error_message = undef;
 
 # output the classes
 foreach my $cmd (@cmd_list)
@@ -262,7 +263,7 @@ foreach my $cmd (@cmd_list)
 	
 	my $cmd_base_class = join(", ", map {"public $_"} @cmd_base_classes);
 	my $cmd_class = $protocol_name."Protocol".$cmd;
-	$cmd_class{$cmd} = $cmd_class;
+	$cmd_classes{$cmd} = $cmd_class;
 	
 	print H <<__E;
 class $cmd_class : $cmd_base_class
@@ -665,8 +666,8 @@ for my $cmd (@cmd_list)
 		my $has_stream = obj_is_type($cmd,'StreamWithCommand');
 		my $argextra = $has_stream?', std::auto_ptr<IOStream> apStream':'';
 		my $queryextra = $has_stream?', apStream':'';
-		my $request_class = $cmd_class{$cmd};
-		my $reply_class = $cmd_class{obj_get_type_params($cmd,'Command')};
+		my $request_class = $cmd_classes{$cmd};
+		my $reply_class = $cmd_classes{obj_get_type_params($cmd,'Command')};
 		
 		print H "\tvirtual std::auto_ptr<$reply_class> Query(const $request_class &rQuery$argextra) = 0;\n";
 		my @a;
@@ -769,8 +770,8 @@ __E
 				my $has_stream = obj_is_type($cmd,'StreamWithCommand');
 				my $argextra = $has_stream?', std::auto_ptr<IOStream> apStream':'';
 				my $queryextra = $has_stream?', apStream':'';
-				my $request_class = $cmd_class{$cmd};
-				my $reply_class = $cmd_class{obj_get_type_params($cmd,'Command')};
+				my $request_class = $cmd_classes{$cmd};
+				my $reply_class = $cmd_classes{obj_get_type_params($cmd,'Command')};
 				print H "\tstd::auto_ptr<$reply_class> Query(const $request_class &rQuery$argextra);\n";
 			}
 		}
@@ -909,7 +910,7 @@ __E
 	{
 		print CPP <<__E;
 	case $cmd_id{$cmd}:
-		return std::auto_ptr<Message>(new $cmd_class{$cmd}());
+		return std::auto_ptr<Message>(new $cmd_classes{$cmd}());
 		break;
 __E
 	}
@@ -1011,9 +1012,9 @@ __E
 		{
 			if(obj_is_type($cmd,'Command'))
 			{
-				my $request_class = $cmd_class{$cmd};
+				my $request_class = $cmd_classes{$cmd};
 				my $reply_msg = obj_get_type_params($cmd,'Command');
-				my $reply_class = $cmd_class{$reply_msg};
+				my $reply_class = $cmd_classes{$reply_msg};
 				my $reply_id = $cmd_id{$reply_msg};
 				my $has_stream = obj_is_type($cmd,'StreamWithCommand');
 				my $argextra = $has_stream?', std::auto_ptr<IOStream> apStream':'';