Some comments.
authorCarl Hetherington <cth@carlh.net>
Mon, 25 Nov 2013 01:03:30 +0000 (01:03 +0000)
committerCarl Hetherington <cth@carlh.net>
Mon, 25 Nov 2013 01:03:30 +0000 (01:03 +0000)
src/lib/config.cc
src/lib/cross.cc

index 2420ab1b5ef6eb637a073db9dbf3a10dc9a34e2f..54b9168f2542a1bbd8ee49d0b933f4c95007ada2 100644 (file)
@@ -181,6 +181,7 @@ Config::read ()
 void
 Config::read_old_metadata ()
 {
+       /* XXX: this won't work with non-Latin filenames */
        ifstream f (file(true).string().c_str ());
        string line;
 
index 9aa8454c989444dd04532a4b9ef995c36c279b20..7436dbf26a9b4582edcfb8e1702638b2e91d3416 100644 (file)
@@ -70,6 +70,9 @@ cpu_info ()
        string info;
        
 #ifdef DCPOMATIC_LINUX
+       /* This use of ifstream is ok; the filename can never
+          be non-Latin
+       */
        ifstream f ("/proc/cpuinfo");
        while (f.good ()) {
                string l;