New Music Browser

Having a reasonable amount of time on my hands and wanting to learn c++, i 
thought i give rewriting the musicbrowser using gtkmm a go thinking the 
basics couldn't be that hard. I have since realised my mistake, however I 
have come up a half finished rewrite.

So here it is, the rather large patch is at http://www- 
users.york.ac.uk/~djh123/newbrowser.diff

there are also a couple of screenshots at http://www- 
users.york.ac.uk/~djh123/screenshot1.png (in a new view mode) and 
http://www-users.york.ac.uk/~djh123/screenshot2.png (in the old 2 pane 
setup)

I havn't patched the autoconf files to check for gtkmm or anything like 
that as i wouldn't know where to start, i just hacked the musicbrowser 
makefile.am to get it to compile on my computer.

Comments, ideas, contributions, etc very much welcome.

Also, i just checked out CVS version, to check the patch worked on it and 
noticed a problem with the playlist plugins meaning it wouldn't complile. 
this fixes that, but isn't exactly an ideal solution to the problem

diff -u -r zinfun/plm/playlist/m3u/m3u.cpp zinf/plm/playlist/m3u/m3u.cpp
--- zinfun/plm/playlist/m3u/m3u.cpp	2003-08-20 01:02:21.000000000 +0000
+++ zinf/plm/playlist/m3u/m3u.cpp	2003-08-21 20:24:46.000000000 +0000
@@ -105,7 +105,7 @@
     string entry;
     bool   removetmp = false;
 -    result = URLToFilePath(url, path);
+    result = URLToFilePath(url.c_str(), path);
 //cerr << "m3u url=" << url << " path=" << path << endl;
 @@ -187,7 +187,7 @@
                     }
                     //cerr << "m3u: newpath=" << path << endl;
                     // make it a url so we can add it to the playlist
-                    FilePathToURL(path,itemurl);
+                    FilePathToURL(path.c_str(),itemurl);
                 }
                         //cerr << "m3u: newurl=" << itemurl << endl;
@@ -229,7 +229,7 @@
 for (vector<string>::iterator i = items.begin();
          i != items.end(); i++) {
-        PlaylistItem *item = new PlaylistItem(*i);
+        PlaylistItem *item = new PlaylistItem((*i).c_str());
         list->push_back(item);
     }
     return r;
diff -u -r zinfun/plm/playlist/pls/pls.cpp zinf/plm/playlist/pls/pls.cpp
--- zinfun/plm/playlist/pls/pls.cpp	2003-08-20 01:02:21.000000000 +0000
+++ zinf/plm/playlist/pls/pls.cpp	2003-08-21 20:34:00.000000000 +0000
@@ -108,7 +108,7 @@
     string file;
     bool   removetmp= false;
 -    Error result = URLToFilePath(url, path);
+    Error result = URLToFilePath(url.c_str(), path);
 if (IsError(result)) {
         // It's a url.. get the local @@ -198,7 +198,7 @@
 for (vector<string>::iterator i = items.begin();
          i != items.end(); i++) {
-        PlaylistItem *item = new PlaylistItem(*i);
+        PlaylistItem *item = new PlaylistItem((*i).c_str());
         list->push_back(item);
     }
     return r;
@@ -344,7 +344,7 @@
             // make it a url so we can add it to the playlist
 -            FilePathToURL(path, urlitem);
+            FilePathToURL(path.c_str(), urlitem);
 }
         items.push_back(urlitem);

-- 
Using M2, Opera's revolutionary e-mail client: http://www.opera.com/m2/


-------------------------------------------------------
This SF.net email is sponsored by: VM Ware
With VMware you can run multiple operating systems on a single machine.
WITHOUT REBOOTING! Mix Linux / Windows / Novell virtual machines
at the same time. Free trial click here:http://www.vmware.com/wl/offer/358/0