On 2019-Nov-01, Fabien COELHO wrote:
> > > #include "common/int.h"
> > > #include "common/logging.h"
> > > -#include "fe_utils/conditional.h"
> > > #include "getopt_long.h"
> > > #include "libpq-fe.h"
> > > +#include "fe_utils/conditional.h"
> > > +#include "fe_utils/cancel.h"
> > > #include "pgbench.h"
> > > #include "portability/instr_time.h"
> >
> > wtf?
>
> I understand that you are unhappy about something, but where the issue is
> fails me, the "wtf" 3 characters are not enough to point me in the right
> direction. Feel free to elaborate a little bit more:-)
I don't see why you move the "conditional.h" line out of its correct
alphabetical position (where it is now), and then add "cancel.h" next to
it also out of its correct alphabetical position.
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services