<div dir="ltr"><div><br></div><div class="gmail_extra"><div class="gmail_quote">On Sat, Sep 7, 2013 at 11:47 PM, Isaac <span dir="ltr"><<a href="mailto:ibid.ag@gmail.com" target="_blank">ibid.ag@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class=""><div class="h5">On Mon, Sep 02, 2013 at 11:15:14AM +0900, Ashwini Sharma wrote:<br>

> HI Rob & list,<br>
><br>
>   Attached is the patch for _route_ command. It does display, add and del<br>
> functions for routing tables.<br>
> Have a look at it.<br>
><br>
> regards,<br>
> Ashwini Kumar<br>
<br>
</div></div>OK, I took a quick look through it.<br>
First, thanks for doing route.<br>
<br>
I note that show_route_help is identical to show_help() apart from the<br>
error message.  That's probably not needed.<br></blockquote><div><br></div>Didn't use show_help(), as this doesn't exit whereas an exit is intended for route.<div>may be show_help() can be updated to do that.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Now...<br>
Hmm, there's an else if (!strcmp) ladder.<br>
It could be reduced to a switch if we take<br>
*argv[0]<<16 + *argv[1]<<8 + *argv[2]<br></blockquote><div><br></div><div>didn't get your logic here, may be a little more detail can help.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<br>
And the help could use a little modification to make it more commonly<br>
understandable:<br>
route [-ne] [-A inet | inet6] [add target | del target]<br>
...<br>
-A inet(6)  Select...<br>
(note one space between the option and its parameter,<br>
but two between the parameter and the explanation)<br></blockquote><div> </div><div>Yes, help message can be updated to the one suggested.  </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<br>
The rest I don't understand well enough to comment on.<br>
<br>
Thanks,<br>
Isaac Dunham<br></blockquote><div><br></div><div>Thanks,</div><div>Ashwini Kumar</div><div> </div></div><br></div></div>