← Back to team overview

maria-developers team mailing list archive

Re: eb121bbe93e: Vanilla cleanups and refactorings

 

Hi, Aleksey!

On Sep 10, Aleksey Midenkov wrote:
> On Fri, Sep 10, 2021 at 9:32 PM Sergei Golubchik <serg@xxxxxxxxxxx> wrote:
> > On Sep 10, Aleksey Midenkov wrote:
> > > revision-id: eb121bbe93e (mariadb-10.6.1-78-geb121bbe93e)
> > > parent(s): d90f54e2206
> > > author: Aleksey Midenkov
> > > committer: Aleksey Midenkov
> > > timestamp: 2021-09-10 17:42:10 +0300
> > > message:
> > >
> > > diff --git a/dbug/tests.c b/dbug/tests.c
> > > index 3e77bf82236..100388891eb 100644
> > > --- a/dbug/tests.c
> > > +++ b/dbug/tests.c
> > > @@ -13,6 +13,17 @@ char *push1=0;
> > >  #include <my_pthread.h>
> > >  #include <string.h>
> > >
> > > +
> > > +#ifndef DBUG_OFF
> > > +#define DBUG_EVALUATE(keyword,a1,a2) \
> > > +        (_db_keyword_(0,(keyword), 0) ? (a1) : (a2))
> > > +#define DBUG_EVALUATE_IF(keyword,a1,a2) \
> > > +        (_db_keyword_(0,(keyword), 1) ? (a1) : (a2))
> > > +#else
> > > +#define DBUG_EVALUATE(keyword,a1,a2) (a2)
> > > +#define DBUG_EVALUATE_IF(keyword,a1,a2) (a2)
> > > +#endif
> > > +
> >
> > could you fix the test properly, please?
> 
> Sorry, I couldn't do that quickly. Do you know the recipe for how this is done?

I thought it's just deleting everything that uses DBUG_EVALUATE*.
Let me try...

I've pushed the commit into bb-10.7-serg - just take it and remove

+#undef DBUG_EVALUATE
+#undef DBUG_EVALUATE_IF
+#define DBUG_IF(X) _db_keyword_(0,X,1)

from tests.c - that'll do.

Regards,
Sergei
VP of MariaDB Server Engineering
and security@xxxxxxxxxxx


Follow ups

References