Skip site navigation (1) Skip section navigation (2)

Re: Add YAML option to explain

From: Itagaki Takahiro <itagaki(dot)takahiro(at)oss(dot)ntt(dot)co(dot)jp>
To: Greg Sabino Mullane <greg(at)turnstep(dot)com>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: Add YAML option to explain
Date: 2009-11-30 03:34:57
Message-ID: 20091130123456.4A03.52131E4D@oss.ntt.co.jp (view raw or flat)
Thread:
Lists: pgsql-hackers
Itagaki Takahiro <itagaki(dot)takahiro(at)oss(dot)ntt(dot)co(dot)jp> wrote:

> My rewrite is relatively large. Please reversely-review the patch.

I rethink the code cleanup should be done with another patch even if needed.
Here is a lite version of yaml YAML explan patch.

All of the logic for indent is done in ExplainYAMLLineStarting().
It can remove undesirable linebreaks from the item list and end of output.

Please check whether the v3.1 patch works as your expectation.
If ok, we can move it to "Ready for Committer".

Regards,
---
ITAGAKI Takahiro
NTT Open Source Software Center


Attachment: yaml.explain.v3.1.patch
Description: application/octet-stream (10.1 KB)

In response to

pgsql-hackers by date

Next:From: Jeff DavisDate: 2009-11-30 05:15:07
Subject: Re: Listen / Notify - what to do when the queue is full
Previous:From: Fujii MasaoDate: 2009-11-30 03:20:53
Subject: Re: Application name patch - v4

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group