some modest improvements in the html rendering of key bindings, plus use of normal...

[ardour.git] / libs / gtkmm2ext / cursors.cc

diff --git a/libs/gtkmm2ext/cursors.cc b/libs/gtkmm2ext/cursors.cc
index 3377a7cda9e5f45a72e0280d064ae9022d1c3c50..f8862835f68af5680197af34d2adccf6178ffa7c 100644 (file)
--- a/libs/gtkmm2ext/cursors.cc
+++ b/libs/gtkmm2ext/cursors.cc
@@ -1,8 +1,32 @@
+/*
+    Copyright (C) 2014 Paul Davis
+
+    This program is free software; you can redistribute it and/or modify
+    it under the terms of the GNU General Public License as published by
+    the Free Software Foundation; either version 2 of the License, or
+    (at your option) any later version.
+
+    This program is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+    GNU General Public License for more details.
+
+    You should have received a copy of the GNU General Public License
+    along with this program; if not, write to the Free Software
+    Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+
+*/
+
 #include <sstream>
-#include <fstream>
+
+#include "pbd/gstdio_compat.h"
+#include "pbd/error.h"
+#include "pbd/compose.h"
 
 #include "gtkmm2ext/cursors.h"
 
+#include "i18n.h"
+
 using namespace Gtkmm2ext;
 
 CursorInfo::Infos CursorInfo::infos;
@@ -17,34 +41,49 @@ CursorInfo::CursorInfo (const std::string& n, int hotspot_x, int hotspot_y)
 int
 CursorInfo::load_cursor_info (const std::string& path)
 {
-        std::ifstream infofile (path.c_str());
-
-        if (!infofile) {
-                return -1;
-        }
+       gchar *buf = NULL;
+       if (!g_file_get_contents (path.c_str(), &buf, NULL, NULL))  {
+               return -1;
+       }
+       std::stringstream infofile (buf);
+       g_free (buf);
 
-        std::stringstream s;
         std::string name;
         int x;
         int y;
+       bool parse_ok;
+       int line_number = 1;
 
         do {
-                s << infofile;
+               parse_ok = false;
+               infofile >> name;
+                if (!infofile) {
+                       /* failing here is OK ... EOF */
+                       parse_ok = true;
+                        break;
+                }
+               infofile >> x;
                 if (!infofile) {
                         break;
                 }
-                s >> name;
-                s >> x;
-                s >> y;
-                if (!s) {
+               infofile >> y;
+                if (!infofile) {
                         break;
                 }
-                
-                CursorInfo* ci = new CursorInfo (name, x, y);
-                infos[name] = ci;
+
+                parse_ok = true;
+               line_number++;
+
+                infos[name] = new CursorInfo (name, x, y);
 
         } while (true);
 
+       if (!parse_ok) {
+               PBD::error << string_compose (_("cursor hotspots info file %1 has an error on line %2"), path, line_number) << endmsg;
+               infos.clear ();
+               return -1;
+       }
+
         return 0;
 }