maria-developers team mailing list archive
-
maria-developers team
-
Mailing list archive
-
Message #08232
Re: Comments/thoughts on patch
great thx! (was on my TODO...but got stuck on other things today).
I'll write a new patch, that passes this test too
/Jonas
On Thu, Mar 5, 2015 at 9:54 AM, Kristian Nielsen <knielsen@xxxxxxxxxxxxxxx>
wrote:
> Jonas Oreland <jonaso@xxxxxxxxxx> writes:
>
> > I wrote attached patch (so that i would understand what code actually
> does)
> > and found myself while refactoring also doing a change, namely that
> > i removed the check for server_id in a couple of places.
> >
> > it still passes all tests...
>
> It is somewhat surprising (and bad) that there were no test cases that
> caught
> this.
>
> So I pushed a suitable test case:
>
> http://lists.askmonty.org/pipermail/commits/2015-March/007520.html
>
> Maybe it can help you...
>
> - Kristian.
>
Follow ups
References