MASON-INTEREST-L Archives

February 2012

MASON-INTEREST-L@LISTSERV.GMU.EDU

Options: Use Proportional Font
Show Text Part by Default
Condense Mail Headers

Message: [<< First] [< Prev] [Next >] [Last >>]
Topic: [<< First] [< Prev] [Next >] [Last >>]
Author: [<< First] [< Prev] [Next >] [Last >>]

Print Reply
Sender:
MASON Multiagent Simulation Toolkit <[log in to unmask]>
Date:
Fri, 24 Feb 2012 19:12:57 -0500
MIME-version:
1.0 (Apple Message framework v1084)
Reply-To:
MASON Multiagent Simulation Toolkit <[log in to unmask]>
Content-type:
text/plain; charset=us-ascii
Subject:
From:
Sean Luke <[log in to unmask]>
In-Reply-To:
Content-Transfer-Encoding:
8bit
Parts/Attachments:
text/plain (21 lines)
On Feb 24, 2012, at 5:51 PM, Matt L. Miller wrote:

> indexAfterArgumentForKey(String key, String[] args, int startingAt)
> keyExists(String key, String[] args, int startingAt)
> argumentForKey(String key, String[] args, int startingAt)
> 
> None of those startingAt arguments appear to do anything (probably because
> nothing in the class ever needs anything other than "0").

Yep, a little cruft there.  Actually on SVn indexAfterArgumentForKey is commented out.
I cleaned it up on SVnN.

> Far more nit-picky: I noticed that the comment in keyExists says "key can't
> be the last string," no doubt a carry over from copying the code from one of
> the other two methods; fortunately, the code itself doesn't reflect this,
> since, of course, -help can be the last key.

Right.

Sean

ATOM RSS1 RSS2