Yes it is ridiculous for now... and maybe this function finaly go into Util.java.<br><br>But there are others parts of codes in the audio section that could be factorized into it, but I didn't have time to push my idea yet...<br>

<br>To be continued... or not...<br><br>Alex<br><br><div class="gmail_quote">2012/8/24 Edward Wang <span dir="ltr"><<a href="mailto:edward.c.wang@compdigitec.com" target="_blank">edward.c.wang@compdigitec.com</a>></span><br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 12-08-24 11:31 AM, <a href="mailto:git@videolan.org" target="_blank">git@videolan.org</a> (Alexandre Perraud) <<a href="mailto:git@videolan.org" target="_blank">git@videolan.org</a> (Alexandre Perraud)> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
new file mode 100644<br>
index 0000000..bafacfa<br>
--- /dev/null<br>
+++ b/vlc-android/src/org/<u></u>videolan/vlc/gui/audio/<u></u>AudioUtil.java<br>
@@ -0,0 +1,55 @@<br>
+/****************************<u></u>******************************<u></u>*******************<br>
+ * AudioUtil.java<br>
+ ********************<br>
</blockquote></div>
Why create a new class? This could just go into Util.java instead of creating new classes. A class for one functions seems a bit ridiculous.<br>
<br>
Regards,<br>
        Edward Wang<div class="HOEnZb"><div class="h5"><br>
______________________________<u></u>_________________<br>
Android mailing list<br>
<a href="mailto:Android@videolan.org" target="_blank">Android@videolan.org</a><br>
<a href="http://mailman.videolan.org/listinfo/android" target="_blank">http://mailman.videolan.org/<u></u>listinfo/android</a><br>
</div></div></blockquote></div><br>