orderedhash_revamp branch is ready for merging

James E Keenan jkeen at verizon.net
Sun Jan 24 03:08:54 UTC 2010


Vasily Chekalkin wrote:
> Hello.
>
> orderedhash_revamp branch is ready for merging. Main purpose of branch -
> reimplement OrderedHash from hackish version of regular Hash (with
> nulling keys, etc) to be first-class citizen.
>
> Testing will be very appreciated.
>
Bacek,

I got a PASS for this on 'make test'.  See: 
http://smolder.plusthree.com/app/public_projects/report_details/31866

However, the 'svn diff' is enormous:  > 1.5 megs compressed; 282766 
lines long.  This is going to be difficult for humans to review (at 
least this one ;-) ) and will probably be a PITA to 'svn merge'.

If you were to apply the essence of your changes to a branch forked more 
recently from trunk, would that yield a more manageable patch?

Thank you very much.

kid51



More information about the parrot-dev mailing list