| 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 |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| 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 | Content-Type | Size |
|---|---|---|
| yaml.explain.v3.1.patch | application/octet-stream | 10.1 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Jeff Davis | 2009-11-30 05:15:07 | Re: Listen / Notify - what to do when the queue is full |
| Previous Message | Fujii Masao | 2009-11-30 03:20:53 | Re: Application name patch - v4 |