Hi guys,
On Tue, 2011-07-19 at 10:13 +0100, Michael Meeks wrote:
> Having said that - I'm not a pure PITA if there is an accurate and
> complete patch of what actually changed here to review - then we can
> review it - but I'm not a fan of approving big, confused, un-quantified
> changes at
Hi Kohei,
On Mon, 2011-07-18 at 16:57 -0400, Kohei Yoshida wrote:
> Just to tone down this statistics a bit. The vast majority of changes
> are in fact code relocations with no changes e.g. the change in
> rectangle_set is purely moving code from one file to another.
:-) Any chance of a
On Mon, 2011-07-18 at 15:53 +0100, Michael Meeks wrote:
> Hi there,
>
> On Fri, 2011-07-15 at 00:55 +0200, Markus Mohrhard wrote:
> > So one review needed to get it into 3.4.2. It is an quite important
> > fix as we got several reports where people reported they thought that
> > calc froze and swi
Hello Michael, Kohei,
I feel extremely uncomfortable (personally) introducing a couple of
> thousands of LOC change between RC1 and final for 3.4.2. We're in a
> triple review mode because our changes will not get much testing before
> they are unleashed on tens of thousands of new people p
Hi there,
On Fri, 2011-07-15 at 00:55 +0200, Markus Mohrhard wrote:
> So one review needed to get it into 3.4.2. It is an quite important
> fix as we got several reports where people reported they thought that
> calc froze and switched back to 3.3.2.
So:
git diff --numstat 0_5_2..0_5_3
On Thu, 2011-07-14 at 10:31 +0200, David Tardon wrote:
> Nitpick, unrelated to the review: you may want to disable operator= for
> storage_filled_linear (and storage_filled_nested_array too), because its
> use would cause double delete.
Good point. Done (on master for the next release). Thanks
It looks sane too me, too. I build with it and tested with some of the test
documents and had a quick look at the patches for 0.5.3. I hope that this is
enough for an external lib.
So one review needed to get it into 3.4.2. It is an quite important fix as
we got several reports where people report
On Wed, Jul 13, 2011 at 11:15:33PM -0400, Kohei Yoshida wrote:
> Hi there,
>
> I'd like to update mdds to 0.5.3 for the -3-4 branch and preferably the
> -3-4-2 branch as well. The attached patches would make that happen.
>
> That should improve the performance issue raised in
>
> https://bugs.f
