Hello,<br><br><div class="gmail_quote">On Mon, May 9, 2011 at 11:06 AM, Jean-Baptiste Kempf <span dir="ltr"><<a href="mailto:git@videolan.org">git@videolan.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
vlc | branch: master | Jean-Baptiste Kempf <<a href="mailto:jb@videolan.org">jb@videolan.org</a>> | Mon May  2 22:40:16 2011 +0200| [c1fd72005df18877637144885a8da01c832a2e5e] | committer: Jean-Baptiste Kempf<br><br>

diff --git a/modules/gui/qt4/components/playlist/selector.hpp b/modules/gui/qt4/components/playlist/selector.hpp<br>
index d97335e..ddac879 100644<br>
--- a/modules/gui/qt4/components/playlist/selector.hpp<br>
+++ b/modules/gui/qt4/components/playlist/selector.hpp<br>
@@ -43,7 +43,8 @@ class QLabel;<br>
 enum SelectorItemType {<br>
     CATEGORY_TYPE,<br>
     SD_TYPE,<br>
-    PL_ITEM_TYPE<br>
+    PL_ITEM_TYPE,<br>
+    SQL_ML<br>
 };<br></blockquote><div><br></div><div>Why not SQL_ML_TYPE for consistency?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">@@ -189,8 +189,11 @@ void StandardPLPanel::searchDelayed( const QString& searchText )<br>

<br>
 /* Set the root of the new Playlist */<br>
 /* This activated by the selector selection */<br>
-void StandardPLPanel::setRoot( playlist_item_t *p_item )<br>
+void StandardPLPanel::setRoot( playlist_item_t *p_item, bool b )<br>
 {<br>
+    if( b ) //SQLML<br>
+        return;<br>
+<br>
     model->rebuild( p_item );<br>
 }<br></blockquote><div><br></div><div>This use of the signal argument (bool) is quite non-transparent. Wouldn't it be better to make another signal for SQL ML?</div><div><br></div><div>Cheers</div></div>