<span class="gmail_quote">On 03 Jan 2008 02:56:57 +0000, <b class="gmail_sendername">N.C. Wilson</b> <<a href="mailto:ncw33@cam.ac.uk">ncw33@cam.ac.uk</a>> wrote:</span><blockquote class="gmail_quote" style="margin-top: 0; margin-right: 0; margin-bottom: 0; margin-left: 0; margin-left: 0.80ex; border-left-color: #cccccc; border-left-width: 1px; border-left-style: solid; padding-left: 1ex">
On Thursday 03 January 2008 1:33 am Leo Franchi wrote:<br>>  you definitely need to comment your code here.<br>> CollectionSortFilterProxyModel::lessThanString needs some doc lovin' to bo<br>> honest a relatively quick readthrough shows me that it would take me quite
<br>> a significant amount of time to parse the code, so i'm going to wait for<br>> some comments.<br><br>Here are the comments (attached). </blockquote><br><blockquote class="gmail_quote" style="margin-top: 0; margin-right: 0; margin-bottom: 0; margin-left: 0; margin-left: 0.80ex; border-left-color: #cccccc; border-left-width: 1px; border-left-style: solid; padding-left: 1ex">
This is quite a nice patch for any listeners of classical music, who will<br>want to be able to easily find numbered pieces of music and expect to find<br>them in the 'right' place. Having Symphony 3 ... Symphony 24 ... Symphony
<br>104 as opposed to the standard order is a nice little feature, and it does<br>not interfere with any pieces that you would not want sorted this way.</blockquote><div><br><div><br>looks good. but this is not a patch! :)
<br><br>also, you should consider joining us on #amarok on <a href="http://irc.freenode.net">irc.freenode.net</a>, where amarok devs and users hang out. we can work through issues much faster in that way too..<br><br>regarding the "temporary" dialog, i'm going to say we should wait until we finalize something and commit that. if we commit something temporarily chances are it will be forgotten (as there are so many things to do in amarok2) and we don't want to ship it when there are much better ways to show the controls (as i elaborated on in my response).
<br></div></div><br><blockquote class="gmail_quote" style="margin-top: 0; margin-right: 0; margin-bottom: 0; margin-left: 0; margin-left: 0.80ex; border-left-color: #cccccc; border-left-width: 1px; border-left-style: solid; padding-left: 1ex">
When you say it would take time to parse, if you mean time to parse regexps<br>at runtime, then probably not. Most of the time the function returns<br>straight away after checking the first string and will usually not even
<br>read the second one before passing control to the standard sort.</blockquote><div><br></div>when i said parse, i meant it takes time for my brain to parse that function without comments :) so we're all set now.<br>
<br>leo<br><br clear="all"><br>-- <br>______________________________________________________<br>Leo Franchi                    <br>4305 Charlemagne Ct         <a href="mailto:lfranchi@gmail.com">lfranchi@gmail.com</a> <br>
Austin                                 cell: (650) 704 3680<br>TX, USA                              home: (650) 329 0125