From 003221bb953aad6adde0c66814f29a13236cb1a5 Mon Sep 17 00:00:00 2001
From: "v.audacity" <v.audacity@47890b92-0858-11df-a26f-8b716316a5bc>
Date: Sat, 17 Jul 2010 00:11:57 +0000
Subject: [PATCH] (minor change)

No need to check passed-in value of parameter that's used only for return value.
---
 src/AutoRecovery.cpp | 4 +---
 src/AutoRecovery.h   | 3 +++
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/AutoRecovery.cpp b/src/AutoRecovery.cpp
index 3181845d8..7ac329584 100644
--- a/src/AutoRecovery.cpp
+++ b/src/AutoRecovery.cpp
@@ -220,9 +220,7 @@ static bool RecoverAllProjects(AudacityProject** pproj)
 bool ShowAutoRecoveryDialogIfNeeded(AudacityProject** pproj,
                                     bool *didRecoverAnything)
 {
-   if (didRecoverAnything)
-      *didRecoverAnything = false;
-
+   *didRecoverAnything = false;
    if (HaveFilesToRecover())
    {
       AutoRecoveryDialog dlg(*pproj);
diff --git a/src/AutoRecovery.h b/src/AutoRecovery.h
index c940418d9..47dc83ee5 100644
--- a/src/AutoRecovery.h
+++ b/src/AutoRecovery.h
@@ -26,6 +26,9 @@
 // Returns: True, if the start of Audacity should continue as normal
 //          False if Audacity should be quit immediately
 //
+// The didRecoverAnything param is strictly for a return value. 
+// Any value passed in is ignored.
+// 
 bool ShowAutoRecoveryDialogIfNeeded(AudacityProject** pproj,
                                     bool *didRecoverAnything);