Hey Sasha,

On Tue, 2008-08-19 at 02:01 +0300, Sasha Khapyorsky wrote:
> On 15:50 Mon 18 Aug     , Al Chu wrote:
> > 
> > The reason is that the ordering of the if statements now matters.  If I
> > add a new command called "DoSomething", it must come after the "Dump"
> > comparison, otherwise the command "D" could take the "DoSomething"
> > branch.
> 
> Sure, order will matter - new commands will be at end.

I also realized that this approach (which is in the patch I posted last
night), would allow typos like "star" and "hel" to work.

Is that ok with you?

Al

> 
> Sasha
> 
> > We can add a comments or something to document this.  It's
> > obviously just a style difference.
> > 
> > > >
> > > > But we would require
> > > > whitespace between the command and options for that to work.
> > > 
> > > This is fine.
> > > 
> > > What I meant is follow:
> > > 
> > >   unsigned cmd_len = 0;
> > > 
> > >   while (isalpha(line[cmd_len]))
> > >           cmd_len++;
> > > 
> > >   if (!strncasecmp(line, "Dump", cmdlen))
> > >           ...
> > > 
> > > In this case strings "D", "du", etc. will be resolved as "Dump" command.
> > 
> > Ok.  I see what you were thinking now.  
> > 
> > Al
> > 
> > > Sasha
> > -- 
> > Albert Chu
> > [EMAIL PROTECTED]
> > 925-422-5311
> > Computer Scientist
> > High Performance Systems Division
> > Lawrence Livermore National Laboratory
> > 
-- 
Albert Chu
[EMAIL PROTECTED]
925-422-5311
Computer Scientist
High Performance Systems Division
Lawrence Livermore National Laboratory

_______________________________________________
general mailing list
[email protected]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to