<div dir="ltr"><div><div><div>This sorting will only by used while loading an album or appending it in current playlist.<br><br></div>I would prefer you to not sort in getLocations, but call Collections.sort in handleContextItemSelected and in OnItemClickListener for album in AudioBrowserFragment.<br><br></div>This should prevent adding too much code in AudioBrowserListAdapter and computing useless checks/sorts.<br><br></div>Otherwise it's all good.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Dec 5, 2014 at 2:50 AM, Enno G. <span dir="ltr"><<a href="mailto:mrmaffen@googlemail.com" target="_blank">mrmaffen@googlemail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hey,<div><br></div><div>setting a default number for the field mTrackNumber in Media.java wouldn't change anything. Setting an empty String in TrackInfo.java is wrong in my opinion.</div><div>So I just added an "if (empty || null)"-check, so that the NumberFormatException is only thrown if the String is non-null, non-empty and not parsable.</div><div><br></div><div>Greets</div><span class="HOEnZb"><font color="#888888"><div>Enno</div></font></span></div>
<br>_______________________________________________<br>
Android mailing list<br>
<a href="mailto:Android@videolan.org">Android@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/android" target="_blank">https://mailman.videolan.org/listinfo/android</a><br>
<br></blockquote></div><br></div>