<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2013/7/25 Holger Hans Peter Freyther <span dir="ltr"><<a href="mailto:holger@freyther.de" target="_blank">holger@freyther.de</a>></span><br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, Jul 25, 2013 at 06:19:24PM +0400, Ivan Kluchnikov wrote:<br>
<br>
> - added 'write' implementation<br>
<br>
please write a VTY test for this routine. It is like 20 lines of<br>
python and will make sure that bitrot is detected early (a perfect<br>
way to secure your time investment).<br>
<br></blockquote><div> </div><div>Yes, I agree, tests are very important, I will try to write one for this command.<br><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<br>
+       if ((bts->si_common.rach_control.t3) != 0)<br>
...<br>
<br>
+       if ((bts->si_common.rach_control.t2 & 0xfb) != 0)<br>
...<br>
<br>
you want to avoid executing the for loop?<br>
<span class=""><font color="#888888"><br></font></span></blockquote><div>Yes, do you think it is redundant part of the code?<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<span class=""><font color="#888888">
<br>
holger<br>
<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br>Regards,<br>Ivan Kluchnikov.<br><a href="http://fairwaves.ru/" target="_blank">http://fairwaves.ru</a><br>

</div></div>