Re: updated emacs configuration - Mailing list pgsql-hackers

From Noah Misch
Subject Re: updated emacs configuration
Date
Msg-id 20130626133537.GA857151@tornado.leadboat.com
Whole thread Raw
In response to Re: updated emacs configuration  (Peter Eisentraut <peter_e@gmx.net>)
Responses Re: updated emacs configuration
List pgsql-hackers
On Tue, Jun 25, 2013 at 11:17:47PM -0400, Peter Eisentraut wrote:
> On Sun, 2013-06-23 at 16:03 -0400, Noah Misch wrote:
> > > ((c-mode . ((c-basic-offset . 4)
> > >             (fill-column . 79)
> > 
> > I don't know whether you'd consider it to fall within the scope of this
> > update, but 78 is the fill-column setting that matches pgindent.
> 
> Well, well, well.  I did some extensive tests some time ago when that
> setting was added.  I have a suspicion that this could be related to the
> recent pgindent changes (which everyone claims were no changes).  I'm
> going to have to research that some more.

pgindent has not changed the following xlog.c comment since April 2011, but
fill-column 77 or 79 changes it:
/* * fullPageWrites is the master copy used by all backends to determine * whether to write full-page to WAL, instead
ofusing process-local one. * This is required because, when full_page_writes is changed by SIGHUP, * we must WAL-log it
beforeit actually affects WAL-logging by backends. * Checkpointer sets at startup or after SIGHUP. */
 

Note that emacs and pgindent remain at odds over interior tabs in comments.
When pgindent finds a double-space (typically after a sentence) ending at a
tab stop, it replaces the double-space with a tab.  c-fill-paragraph will
convert that tab to a *single* space, and that can be enough to change many
line break positions.

-- 
Noah Misch
EnterpriseDB                                 http://www.enterprisedb.com



pgsql-hackers by date:

Previous
From: Andrew Dunstan
Date:
Subject: Re: Kudos for Reviewers -- straw poll
Next
From: Amit Kapila
Date:
Subject: Re: Optimizing pglz compressor