[mythtv] [PATCH] for mythmusic...

[DanM]

Christian, sorry for comming off a little strong.  Thanks for fixing it 
:-)  Sorry for the HTML mail, thought I had snowman.net in my text only 
list.  Will have to update my mozilla at the house.

-dan

Christian Hoenig wrote:

>-----BEGIN PGP SIGNED MESSAGE-----
>Hash: SHA1
>
>Hi,
>
>First of all, please don't send HTML mail to a Mailinglist!
>
>DanM wrote
>  
>
>>Why did you break existing functionality?    
>>    
>>
>
>I didn't think I'd break anything. It was quater past 3 am ;-).
>
>  
>
>>More
>>specificly, I'm refering to:<br>
>><pre wrap="">-    allowed_modes = QStringList::split(",", visualname);
>>-
>>-    if (allowed_modes.contains("Random"))
>></pre>
>>- if (allowed_modes.size() &gt; 1)<br>
>><pre wrap="">-            vis_mode_index = rand() % allowed_modes.size();
>></pre>
>>- current_visual_name =
>>allowed_modes[vis_mode_index].stripWhiteSpace();<br> <br>
>>You removed the ability to have a pre-defined list of visualizers to be
>>randomly selected from.<br>
>>    
>>
>
>I'll fix that later tonight, thanks for the suggestion.
>
>take care, have fun
>/christian
>-----BEGIN PGP SIGNATURE-----
>Version: GnuPG v1.2.2 (GNU/Linux)
>
>iD8DBQE/JOFwmauZIX3ydQURApoiAJ9MPtIXWQAqktEnudX0sGnMSfYVvQCg8ibm
>ValHE0aq9B+/DvuP4xyFM/M=
>=wtbM
>-----END PGP SIGNATURE-----
>
>
>_______________________________________________
>mythtv-dev mailing list
>mythtv-dev at snowman.net
>http://lists.snowman.net/cgi-bin/mailman/listinfo/mythtv-dev
>  
>