diff -r a2467631ae02 -r 1f002146abb4 telutils/keysequencerecognitionservice/src/simcontrolkeysequencehandler.cpp
--- a/telutils/keysequencerecognitionservice/src/simcontrolkeysequencehandler.cpp	Wed Jun 23 18:49:32 2010 +0300
+++ b/telutils/keysequencerecognitionservice/src/simcontrolkeysequencehandler.cpp	Tue Jul 06 14:53:02 2010 +0300
@@ -73,7 +73,9 @@
 }
 
 
-
+/*!
+  SimControlKeySequenceHandler::parseString.
+ */
 bool SimControlKeySequenceHandler::parseString(const QString &keySequence)
 {
     DPRINT_METHODENTRYEXIT;
@@ -88,7 +90,7 @@
     QString verifyNewPin;
     QString puk;
     
-    SimOperation operation;
+    SimOperation operation = None;
     bool handled = false;
    
     QString keySequencePrefix (keySequence);
@@ -150,8 +152,11 @@
     return handled;
 }
 
-void SimControlKeySequenceHandler::processChangePin(SimOperation operation, QString oldPin,
-                                                            QString newPin, QString verifyNew)
+/*!
+  SimControlKeySequenceHandler::processChangePin.
+ */
+void SimControlKeySequenceHandler::processChangePin(SimOperation operation, const QString &oldPin,
+                                                   const QString &newPin, const QString &verifyNew)
     {
     CManualSecuritySettings::TPin pin;
     
@@ -171,9 +176,11 @@
        QT_TRAP_THROWING(m_securityModel->ChangePinL(pin, oldPinBuf, newPinBuf, verifyNewBuf));
     }
 
-
-void SimControlKeySequenceHandler::processUnblockPin(SimOperation operation, QString puk, QString newPin, 
-                                                                                 QString verifyNew)
+/*!
+  SimControlKeySequenceHandler::processUnblockPin.
+ */
+void SimControlKeySequenceHandler::processUnblockPin(SimOperation operation, const QString &puk, 
+                                                   const QString &newPin, const QString &verifyNew)
     {
     CManualSecuritySettings::TPin pin;