On 4.1.2013 17:42, Robert Haas wrote:
> On Mon, Dec 31, 2012 at 11:51 AM, Tomas Vondra <tv(at)fuzzy(dot)cz> wrote:
>> I thought I followed the conding style - which guidelines have I broken?
> 
> +	/* If there are no non-local relations, then we're done. Release the memory
> +	 * and return. */
> 
> Multi-line comments should start with a line containing only /* and
> end with a line containing only */.
> 
> +DropRelFileNodeAllBuffers(RelFileNodeBackend * rnodes, int nnodes)
> and
> +rnode_comparator(const void * p1, const void * p2)
> 
> The extra spaces after the asterisks should be removed.
> 
> +void smgrdounlinkall(SMgrRelation * rels, int nrels, bool isRedo)
> +{
> 
> void should be on a line by itself.
> 
> Sorry to nitpick.

No, thanks for the nitpicking! Code style is important.

> As for BSEARCH_LIMIT, I don't have a great idea - maybe just
> DROP_RELATIONS_BSEARCH_LIMIT?

Sounds good. I've changed the name and fixed the codestyle issues in the
attached version of the patch.

Tomas

In response to

• Re: PATCH: optimized DROP of multiple tables within a transaction at 2013-01-04 16:42:07 from Robert Haas

Responses

• Re: PATCH: optimized DROP of multiple tables within a transaction at 2013-01-07 09:07:39 from Shigeru Hanada

pgsql-hackers by date

Next:	From: Tomas Vondra	Date: 2013-01-06 04:03:38
	Subject: Re: PATCH: Split stats file per database WAS: autovacuum stress-testing our system
Previous:	From: Tomas Vondra	Date: 2013-01-06 03:13:12
	Subject: Re: too much pgbench init output