MASON-INTEREST-L Archives

March 2016

MASON-INTEREST-L@LISTSERV.GMU.EDU

Options: Use Monospaced Font
Show Text Part by Default
Show All Mail Headers

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

Print Reply
Subject:
From:
Sean Luke <[log in to unmask]>
Reply To:
MASON Multiagent Simulation Toolkit <[log in to unmask]>
Date:
Thu, 17 Mar 2016 11:18:11 -0400
Content-Type:
text/plain
Parts/Attachments:
text/plain (19 lines)
Yeah, that's a documentation error.  But before I change it, any opinions?

SimpleProperties is kind of in flux.  I added the ability to sort the properties arbitrarily (not documented), and the sort based on an array of strings representing property names (the poorly named makeFilterComparator(...) -- I will change that method name).  And the ability to sort alphabetically.  And of course no sorting.

Originally I was going to make alphabetical sorting the default.  But I've since found that alphabetical ordering can be a real pain.  People put methods in their source code in a specific order *on purpose*, and alphabetical ordering totally messes with that.  So I removed the alphabetical ordering default but kept it as an option.  What's your opinion on this?

Sean

On Mar 17, 2016, at 5:21 AM, Christian Meyer <[log in to unmask]> wrote:

> Hi,
> 
> I just noticed that SimpleProperties gets initialized without a 
> sortComparator (null), although documentation states that it sorts 
> alphabetically by default.
> 
> Best regards,
> Christian

ATOM RSS1 RSS2