Clarification of DateStyle

Bruce Momjian <pgman@candle.pha.pa.us> writes:

I have applied the following patch to clarify the meaning of the second
DateStyle field.

Am I seeing correctly that this is actually just a one-word insertion?
You do your readers no favor by providing patches for comment that
consist largely of content-free rearrangement of line breaks.

regards, tom lane

Tom Lane wrote:

Bruce Momjian <pgman@candle.pha.pa.us> writes:

I have applied the following patch to clarify the meaning of the second
DateStyle field.

Am I seeing correctly that this is actually just a one-word insertion?
You do your readers no favor by providing patches for comment that
consist largely of content-free rearrangement of line breaks.

Yea, confusing. The change was from:

and the date field order specification

to:

and the input/output specification for year/month/day ordering

When I am in there, I take the opportunity to clean up the arrangement.
If I don't my new text pushes everything out of alignment.

I guess I could make the changes, do a diff, then reformat before
applying to CVS. Would that help?

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073

Bruce Momjian <pgman@candle.pha.pa.us> writes:

I guess I could make the changes, do a diff, then reformat before
applying to CVS. Would that help?

Yeah, it'd make it easier to see the forest for the trees...

regards, tom lane