Hi Dieter,<br><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">> Hi Michal,<br>
><br>
> I did related modifications on converting PMA_CommonFunctions class to a<br>
> static class. Now that class has renamed as PMA_Util. My pull request can be<br>
> found at [0].<br>
><br>
> [0] : <a href="https://github.com/phpmyadmin/phpmyadmin/pull/91" target="_blank">https://github.com/phpmyadmin/phpmyadmin/pull/91</a><br>
<br>
</div>Patches look good (apart from a cosmetic remark, see comment in<br>
gituhub), OK for me to merge.</blockquote><div><br></div><div>First, thanks for reviewing it soon. :)</div><div>I feel I need more clarification on the comment before doing modifications. Are you mentioning about the new lines just after starting brace of functions (all functions) ?</div>
<div>In my point, I feel if any function have very few lines (less than 10 or like that), no need of empty line at the just after starting brace of that function. But for others the empty line will help to improve readability.</div>
<div><br></div><div>Please let me know am I misunderstood your point. And if I modify this, do I need to again make a pull request as I did before ? (That was my first pull request :) )</div><div><br></div><div>Regards !</div>
</div><div>-- </div><font size="4">Chanaka Dharmarathna<br></font><div><font color="#999999"><b>Virtusa (Pvt) Ltd. | </b></font><b style="color:rgb(153,153,153)">Sri Lanka</b></div><br>