The libraries proposal process and containers
fox at ucw.cz
Fri Sep 24 13:48:09 EDT 2010
> Firstly, you shouldn't be committing into the containers library from
> your working copy. You're not the maintainer.
> Secondly, if you have patches to propose, you have to:
> * prepare a single patch for review by this list.
> * propose it, via a ticket, and attach the patch.
> * send the patch to this list for comments.
> * after the time is up, if there was consensus, you can recommend to Ian it be pushed.
> Never, ever just push your private code directly to the containers
> So, for example, you should have a proposal to rewrite all the balance
> functions, and a patch that shows what you change. If we all agree, then
> Ian or Simon can commit it.
Sorry. The rewrite process of the balance functions was in tickets 4311
Once again, I am sorry, everyone.
I was feeling urged by the INLINE => INLINABLE stuff and committed
> Ian : can you fix this?
> -- Don
> > Hi,
> > I am terribly sorry if I did it wrong.
> > I posted the tickets to libraries, there were some discussion about some
> > of them.
> > After sorting the INLINE => INLINABLE issue (I agree that in a hurry)
> > I pushed all the tickets (I was working on top of my repo).
> > Once again, if I should have waited more, sorry. Rollback if
> > I overstepped my authority.
> > Sorry,
> > Milan
> > >
> > > Milan just pushed all the following patches directly to the containers
> > > repo, and closed all the tickets associated with the effort.
> > >
> > > * Fix warnings in Data.Map and Data.Set.
> > > * Finish the started worker/wrapper transformation.
> > > * Merge all the OPTIONS and LANGUAGE module pragmas.
> > > * Remove most INLINE from Map, Set, IntMap and IntSet.
> > > * Comment tests and benchmarks on foldlWithKey'
> > > * Worker/wrapper transformation for Data.IntSet.
> > > * Compile only the benchmark source, not the Data/*.hs.
> > > * Add criterion-based benchmark for IntSet.hs
> > > * Add a testsuite for Data.IntSet.
> > > * Further improve Data.Set balance function
> > > * Further improve Data.Map balance function
> > > * Changing delta to 3 in Data.Set.
> > > * Changing delta to 3 in Data.Map.
> > > * Correct Data.Set Arbitrary instance never to return unbalanced trees.
> > > * Correct Data.Map Arbitrary instance never to return unbalanced trees.
> > > * Improve Data.Set benchmark.
> > > * Improve benchmark infrastructure and Data.Map benchmark
> > > * Improve the performance of Data.Set balance function
> > > * Improve the performance of Data.Map balance function.
> > > * Improve performance of Data.Set union and difference operations
> > > * Improve performance of Data.Map union* and difference operations
> > > * Make the Set store the elements evaluated (bang added).
> > >
> > > Have *any* of these patches been proposed for review? containers is a
> > > critical library, and under libraries@ maintainance. This is not what
> > > you do prior to a release.
> > >
> > > Ian, Simon M.: did you get a chance to sign off on this?
> > >
> > > -- Don
> > >
> > > _______________________________________________
> > > Libraries mailing list
> > > Libraries at haskell.org
> > > http://www.haskell.org/mailman/listinfo/libraries
> > >
More information about the Libraries