From 83cc6c0835c613a961b48540d0c85d429c0a275e Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Sun, 28 Feb 2016 11:38:22 +0100 Subject: [PATCH] db/simple: use second Error instance in Open() Don't need to clear the caller-specified one. --- src/db/plugins/simple/SimpleDatabasePlugin.cxx | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/db/plugins/simple/SimpleDatabasePlugin.cxx b/src/db/plugins/simple/SimpleDatabasePlugin.cxx index 326200a6b..b0ac24c20 100644 --- a/src/db/plugins/simple/SimpleDatabasePlugin.cxx +++ b/src/db/plugins/simple/SimpleDatabasePlugin.cxx @@ -208,11 +208,11 @@ try { borrowed_song_count = 0; #endif - if (!Load(error)) { - delete root; + Error error2; + if (!Load(error2)) { + LogError(error2); - LogError(error); - error.Clear(); + delete root; if (!Check(error)) return false;