From 3fe669f547f9af4cb972510eae9c68772b094ea6 Mon Sep 17 00:00:00 2001
From: ocornut <omarcornut@gmail.com>
Date: Sun, 4 Jan 2015 21:37:49 +0000
Subject: [PATCH] Failure in LogToFile() treated at an error (assert) in the
 absence of another type of error handling.

---
 imgui.cpp | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/imgui.cpp b/imgui.cpp
index 1bd32dd0..fe4a27f2 100644
--- a/imgui.cpp
+++ b/imgui.cpp
@@ -233,7 +233,6 @@
  - settings: write more decent code to allow saving/loading new fields
  - settings: api for per-tool simple persistent data (bool,int,float) in .ini file
  - log: LogButtons() options for specifying depth and/orhiding depth slider
- - log: LogTofile() error handling
  - log: have more control over the log scope (e.g. stop logging when leaving current tree node scope)
  - log: be able to right-click and log a window or tree-node into tty/file/clipboard / generalized context menu?
  - filters: set a current filter that tree node can automatically query to hide themselves
@@ -3515,8 +3514,13 @@ void ImGui::LogToFile(int max_depth, const char* filename)
     if (!filename)
         filename = g.IO.LogFilename;
 
-    g.LogEnabled = true;
     g.LogFile = fopen(filename, "ab");
+    if (!g.LogFile)
+    {
+        IM_ASSERT(g.LogFile != NULL); // Consider this an error
+        return;
+    }
+    g.LogEnabled = true;
     g.LogStartDepth = window->DC.TreeDepth;
     if (max_depth >= 0)
         g.LogAutoExpandMaxDepth = max_depth;