diff --git a/src/cmd_do.cpp b/src/cmd_do.cpp
index 48c0877..cf99c59 100644
--- a/src/cmd_do.cpp
+++ b/src/cmd_do.cpp
@@ -9,6 +9,7 @@ using namespace std;
 cmdEngine *CMDENGINE;
 int Cmd_do::check_cmd(const string& _CMD)
 {
+	cout << "inside cmd_do check \n";
 	stack Stack;
 	istringstream sin(_CMD);
 	string word;
@@ -42,7 +43,9 @@ int Cmd_do::check_cmd(const string& _CMD)
 			}
 			else //stack is filled, we need to accept this "[" as SUBCMD 
 			{
+				cout << "word [ :";
 				SUBCMD += word + " ";
+				cout << SUBCMD << ":\n";
 			}
 			Stack.push(word);
 		}
@@ -61,7 +64,9 @@ int Cmd_do::check_cmd(const string& _CMD)
 				}
 				else//stack not empty so we need to accept "]" into SUBCMD
 				{
+					cout << "word ] :";
 					SUBCMD += word + " ";
+					cout << SUBCMD << "\n";
 				}
 			}
 			else// wrong positions 
@@ -72,12 +77,21 @@ int Cmd_do::check_cmd(const string& _CMD)
 		}
 		else if (word == "<")
 		{
-			readUI = 1;
+		//	readUI = 1;
+			if (!Stack.isEmpty())//if stack is not empty, we need to read it
+			{
+				readSUBCMD = 1;
+				SUBCMD += word+" ";
+			}
+			else//stack is empty, root command
+			{
+				readUI = 1;
+			}
 			Stack.push(word);
 		}
 		else if (word == ">")
 		{
-			if (!Stack.isEmpty())
+			if (!Stack.isEmpty())//if stack is not empty
 			{
 				Stack.pop();
 				if (Stack.isEmpty())
@@ -89,6 +103,12 @@ int Cmd_do::check_cmd(const string& _CMD)
 						readUI = false;
 					}
 				}
+				else//stack is not empty, so it mean we are still reading
+				{
+					cout << "word > :";
+					SUBCMD += word + " ";
+					cout << SUBCMD << ":\n";
+				}
 			}
 			else
 			{
@@ -99,6 +119,7 @@ int Cmd_do::check_cmd(const string& _CMD)
 		else if (readSUBCMD == 1 && readUI == 0)
 		{
 			SUBCMD += word + " ";
+			cout << "SUBCMD :" << SUBCMD << ": (stack empty : "<<Stack.isEmpty()<<"\n";
 		}
 		else if (readUI == 1)
 			USERINPUT += word;
diff --git a/src/cmdengine.cpp b/src/cmdengine.cpp
index 8584e48..7319cc8 100644
--- a/src/cmdengine.cpp
+++ b/src/cmdengine.cpp
@@ -28,7 +28,7 @@ void cmdEngine::start_cmdEngine() {
 		cout << "proccessed : " << cmd << "\n";
 		int x = check(cmd);
 		if (x == 0)
-			cout << "\n Error!!! \n";
+			cout << "\n check failed for "<<cmd<<"\n";
 	}
 }
 void cmdEngine::process(string& _CMD) {
@@ -53,7 +53,7 @@ void cmdEngine::process(string& _CMD) {
 			}
 		}
 	}//for loop [0 to _CMD.size()-1 ]
-	cout <<"processing done :"<< _CMD <<"|"<<endl;
+	cout <<"processing done :"<< _CMD <<"<"<<endl;
 }
 int cmdEngine::check(const string& _CMD,int toExecute)//toExecute set true by default
 {
@@ -61,6 +61,7 @@ int cmdEngine::check(const string& _CMD,int toExecute)//toExecute set true by de
 	string major_command;
 	int isCorrect = 0;
 	sin >> major_command;//read first major command , note that submajor can also be read 
+	cout << "Major is : " << major_command << "\n";
 	if (major_command == "read" || major_command == "write"||major_command=="-q") 
 	{
 		//cout << "detected major_command\n";
@@ -76,7 +77,7 @@ int cmdEngine::check(const string& _CMD,int toExecute)//toExecute set true by de
 			if (cmd_io != NULL)
 				delete cmd_io;
 			isCorrect = 0;
-			cout << "\n ERROR : error in command \n ";
+			cout << "\n ERROR : error in command ("<<major_command<<")\n ";
 			return 0;//check failed
 		}
 		cout << "deleting cmd-io\n";
@@ -97,7 +98,9 @@ int cmdEngine::check(const string& _CMD,int toExecute)//toExecute set true by de
 			}
 			return isCorrect;
 		}
+		cout << "initializing cmd_do\n";
 		Cmd_do *cmd_do=new Cmd_do;
+		cout << "calling check_cmd(_CMD) , _CMD is :" << _CMD << ":\n";
 		if(cmd_do->check_cmd(_CMD))
 		{	
 			if(toExecute)
@@ -105,8 +108,9 @@ int cmdEngine::check(const string& _CMD,int toExecute)//toExecute set true by de
 			isCorrect=1;
 		}
 		else
-		{	isCorrect=0;
-			cout<<"\n ERROR : error in command \n";
+		{	
+			isCorrect=0;
+			cout<<"\n ERROR : error in command do\n";
 		}
 		if(cmd_do!=NULL)
 			delete cmd_do;