Tomas Vondra wrote: > >+ values[Anum_pg_mv_statistic_stamcv - 1] = PointerGetDatum(data); > > > >Why the double space (that's actually in several places in several of > >the patches). > > To align the whole block like this: > > nulls[Anum_pg_mv_statistic_stadeps -1] = true; > nulls[Anum_pg_mv_statistic_stamcv -1] = true; > nulls[Anum_pg_mv_statistic_stahist -1] = true; > nulls[Anum_pg_mv_statistic_standist -1] = true; > > But I won't fight for this too hard, if it breaks rules somehow.
Yeah, it will be undone by pgindent. I suggest you pgindent all the patches in the series. With some clever patch vs. patch -R application, you can do it without having to resolve any conflicts when pgindent modifies code that a patch further up in the series modifies again. -- Álvaro Herrera http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers