Remove BackupStoreContext::ReceivedFinishCommand

Replace with CleanUp(). We don't need two methods that do almost the same
thing.

lib/backupstore/BackupCommands.cpp

@@ -226,7 +226,7 @@ std::auto_ptr<BackupProtocolMessage> BackupProtocolFinished::DoCommand(BackupPro
 		"(name=" << rContext.GetAccountName() << ")");
 
 	// Let the context know about it
-	rContext.ReceivedFinishCommand();
+	rContext.CleanUp();
 
 	return std::auto_ptr<BackupProtocolMessage>(new BackupProtocolFinished);
 }

lib/backupstore/BackupStoreContext.cpp

@@ -66,7 +66,7 @@ BackupStoreContext::BackupStoreContext(int32_t ClientID,
   mSaveStoreInfoDelay(STORE_INFO_SAVE_DELAY),
   mpTestHook(NULL)
 // If you change the initialisers, be sure to update
-// BackupStoreContext::ReceivedFinishCommand as well!
+// BackupStoreContext::CleanUp as well!
 {
 }
 
@@ -110,28 +110,13 @@ void BackupStoreContext::CleanUp()
 	// Make sure the store info is saved, if it has been loaded, isn't read only and has been modified
 	if(mapStoreInfo.get() && !(mapStoreInfo->IsReadOnly()) &&
 		mapStoreInfo->IsModified())
-	{
-		mapStoreInfo->Save();
-	}
-}
-
-
-// --------------------------------------------------------------------------
-//
-// Function
-//		Name:    BackupStoreContext::ReceivedFinishCommand()
-//		Purpose: Called when the finish command is received by the protocol
-//		Created: 16/12/03
-//
-// --------------------------------------------------------------------------
-void BackupStoreContext::ReceivedFinishCommand()
-{
-	if(!mReadOnly && mapStoreInfo.get())
 	{
 		// Save the store info, not delayed
 		SaveStoreInfo(false);
 	}
 
+	ReleaseWriteLock();
+
 	// Just in case someone wants to reuse a local protocol object,
 	// put the context back to its initial state.
 	mProtocolPhase = BackupStoreContext::Phase_Version;

lib/backupstore/BackupStoreContext.h

@@ -54,8 +54,6 @@ class BackupStoreContext
 	BackupStoreContext(const BackupStoreContext &rToCopy); // no copying
 
 public:
-
-	void ReceivedFinishCommand();
 	void CleanUp();
 	int32_t GetClientID() {return mClientID;}