changeset 2400:4daf4fa409e0 trunk

[svn] - make a note that hashtable collisions in the plugin matrix are not only possible, but happening, and have strange side effects.
author nenolod
date Thu, 25 Jan 2007 01:52:51 -0800
parents 9b7b11176d4e
children c4c24ca99c31
files ChangeLog src/audacious/pluginenum.c
diffstat 2 files changed, 14 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/ChangeLog	Wed Jan 24 18:40:07 2007 -0800
+++ b/ChangeLog	Thu Jan 25 01:52:51 2007 -0800
@@ -1,3 +1,11 @@
+2007-01-25 02:40:07 +0000  Yoshiki Yazawa <yaz@cc.rim.or.jp>
+  revision [3808]
+  - should not pass null fd to input plugins.
+  
+  trunk/src/audacious/input.c |    3 +++
+  1 file changed, 3 insertions(+)
+
+
 2007-01-24 23:15:46 +0000  Derek Pomery <nemo@m8y.org>
   revision [3806]
   Strip 0xFEFF 
--- a/src/audacious/pluginenum.c	Wed Jan 24 18:40:07 2007 -0800
+++ b/src/audacious/pluginenum.c	Thu Jan 25 01:52:51 2007 -0800
@@ -277,6 +277,12 @@
         return;
     }
 
+    /*
+     * FIXME: this collides in many cases. this is really bad.
+     * i have no idea why we use a hashtable here, there is no point, when
+     * a list of enabled plugins would realistically offer the same 
+     * scalability.                                       - nenolod
+     */
     plugin_matrix = g_hash_table_new_full(g_str_hash, g_int_equal, g_free,
                                           NULL);