[COMMITTERS] pgsql: Fix Perl copyright script to skip .git subdirectory; running it

2012-01-01 Thread Bruce Momjian
Fix Perl copyright script to skip .git subdirectory; running it on those files corrupts the index. Branch -- master Details --- http://git.postgresql.org/pg/commitdiff/9f60880207c6c7d00a2e4e4d904ef720de58f45b Modified Files -- src/tools/copyright.pl |9 - 1 files

[COMMITTERS] pgsql: Don't use tabs in Perl scripts, for consistency.

2012-01-01 Thread Bruce Momjian
Don't use tabs in Perl scripts, for consistency. Branch -- master Details --- http://git.postgresql.org/pg/commitdiff/6ba139dc5a8a5cce42f0508c14a8b42f4fb6b1cc Modified Files -- src/tools/copyright.pl |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) -- Sent via

Re: [COMMITTERS] pgsql: Update copyright notices for year 2012.

2012-01-01 Thread Tom Lane
Bruce Momjian writes: > Update copyright notices for year 2012. It appears to me that this patch broke both of these files: > src/bin/pgevent/MSG1.bin | Bin 28 -> 29 bytes > src/port/win32.ico | Bin 22486 -> 22487 bytes At least, eyeba

Re: [COMMITTERS] pgsql: Fix Perl copyright script to skip .git subdirectory; running it

2012-01-01 Thread Andrew Dunstan
On 01/01/2012 05:59 PM, Bruce Momjian wrote: Fix Perl copyright script to skip .git subdirectory; running it on those files corrupts the index. This patch looks a bit dubious. Instead of: if ($File::Find::name =~ m{^\./\.git$}) I'd use: if ($_ eq '.git') cheers andrew -- Sent

Re: [COMMITTERS] pgsql: Fix Perl copyright script to skip .git subdirectory; running it

2012-01-01 Thread Bruce Momjian
On Sun, Jan 01, 2012 at 07:05:24PM -0500, Andrew Dunstan wrote: > > > On 01/01/2012 05:59 PM, Bruce Momjian wrote: > >Fix Perl copyright script to skip .git subdirectory; running it on > >those files corrupts the index. > > > This patch looks a bit dubious. Instead of: > >if ($File::Find:

[COMMITTERS] pgsql: Skip 'ico' and 'bin' extensions in copyright changes.

2012-01-01 Thread Bruce Momjian
Skip 'ico' and 'bin' extensions in copyright changes. Branch -- master Details --- http://git.postgresql.org/pg/commitdiff/f4cd747a4d760600ec9791a95957e993c320ba63 Modified Files -- src/tools/copyright.pl |1 + 1 files changed, 1 insertions(+), 0 deletions(-) -- Sent vi

Re: [COMMITTERS] pgsql: Fix Perl copyright script to skip .git subdirectory; running it

2012-01-01 Thread Tom Lane
Bruce Momjian writes: > On Sun, Jan 01, 2012 at 07:05:24PM -0500, Andrew Dunstan wrote: >> I'd use: >> if ($_ eq '.git') > The question was whether we wanted to skip non-top-of-tree .git > directories. I wasn't sure we wanted to do that. Do we? I'd think it'd be reasonable to do so. It's not

[COMMITTERS] pgsql: Revert binary change in copyright year adjustment.

2012-01-01 Thread Bruce Momjian
Revert binary change in copyright year adjustment. Branch -- master Details --- http://git.postgresql.org/pg/commitdiff/b5eb06a22a5a811cbdd065ac52580bbcc59469a0 Modified Files -- src/bin/pgevent/MSG1.bin | Bin 29 -> 28 bytes 1 files changed, 0 insertions(+), 0 deletions(

[COMMITTERS] pgsql: Revert binary change in copyright year adjustment.

2012-01-01 Thread Bruce Momjian
Revert binary change in copyright year adjustment. Branch -- master Details --- http://git.postgresql.org/pg/commitdiff/1358801b7a7afed840b780d7c52e20921a483ed1 Modified Files -- src/port/win32.ico | Bin 22487 -> 22486 bytes 1 files changed, 0 insertions(+), 0 deletions(-)

Re: [COMMITTERS] pgsql: Update copyright notices for year 2012.

2012-01-01 Thread Bruce Momjian
On Sun, Jan 01, 2012 at 06:42:27PM -0500, Tom Lane wrote: > Bruce Momjian writes: > > Update copyright notices for year 2012. > > It appears to me that this patch broke both of these files: > > > src/bin/pgevent/MSG1.bin | Bin 28 -> 29 bytes > > src/port/win32.ico

Re: [COMMITTERS] pgsql: Fix Perl copyright script to skip .git subdirectory; running it

2012-01-01 Thread Bruce Momjian
On Sun, Jan 01, 2012 at 07:24:40PM -0500, Bruce Momjian wrote: > On Sun, Jan 01, 2012 at 07:05:24PM -0500, Andrew Dunstan wrote: > > > > > > On 01/01/2012 05:59 PM, Bruce Momjian wrote: > > >Fix Perl copyright script to skip .git subdirectory; running it on > > >those files corrupts the index. >

[COMMITTERS] pgsql: Skip any .git directory for copyright changes, not just top-leve

2012-01-01 Thread Bruce Momjian
Skip any .git directory for copyright changes, not just top-level .git directories. Per suggestion from Andrew Dunstan. Branch -- master Details --- http://git.postgresql.org/pg/commitdiff/bed762c81e6a7f62695d6c8acb78f15c8e85342e Modified Files -- src/tools/copyright.pl |

Re: [COMMITTERS] pgsql: Fix Perl copyright script to skip .git subdirectory; running it

2012-01-01 Thread Bruce Momjian
On Sun, Jan 01, 2012 at 07:41:46PM -0500, Tom Lane wrote: > Bruce Momjian writes: > > On Sun, Jan 01, 2012 at 07:05:24PM -0500, Andrew Dunstan wrote: > >> I'd use: > >> if ($_ eq '.git') > > > The question was whether we wanted to skip non-top-of-tree .git > > directories. I wasn't sure we want

[COMMITTERS] pgsql: Update copyright git skip comment.

2012-01-01 Thread Bruce Momjian
Update copyright git skip comment. Branch -- master Details --- http://git.postgresql.org/pg/commitdiff/6b6137e4efebcd767a349099b3e048fbc7755cca Modified Files -- src/tools/copyright.pl |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) -- Sent via pgsql-committers

[COMMITTERS] pgsql: Use mutex hint bit in PPC LWARX instructions, where possible.

2012-01-01 Thread Tom Lane
Use mutex hint bit in PPC LWARX instructions, where possible. The hint bit makes for a small but measurable performance improvement in access to contended spinlocks. On the other hand, some PPC chips give an illegal-instruction failure. There doesn't seem to be a completely bulletproof way to tel

[COMMITTERS] pgsql: Use LWSYNC in place of SYNC/ISYNC in PPC spinlocks, where possib

2012-01-01 Thread Tom Lane
Use LWSYNC in place of SYNC/ISYNC in PPC spinlocks, where possible. This is allegedly a win, at least on some PPC implementations, according to the PPC ISA documents. However, as with LWARX hints, some PPC platforms give an illegal-instruction failure. Use the same trick as before of assuming th

[COMMITTERS] pgsql: Use 4-byte slock_t on both PPC and PPC64.

2012-01-01 Thread Tom Lane
Use 4-byte slock_t on both PPC and PPC64. Previously we defined slock_t as 8 bytes on PPC64, but the TAS assembly code uses word-wide operations regardless, so that the second word was just wasted space. There doesn't appear to be any performance benefit in adding the second word, so get rid of i