Re: Re: [PATCHES] s_lock.h cleanup

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Re: [PATCHES] s_lock.h cleanup
Date: 2001-01-19 16:40:54
Message-ID: 200101191640.LAA23982@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

> Bruce Momjian writes:
>
> > In looking at the VAX ASM problem, I realized that the ASM in s_lock.h
> > is all formatted differently, making it even more confusing. I have
> > applied the following patch to s_lock.h to try and clean it up.
>
> I don't believe in this patch at all. It makes the assumption that all
> assemblers have equally forgiving lexical rules as a certain subset of
> said assemblers. For example, the VAX code does not look at all like the
> one back when it still worked.

I agree the VAX code was changed in the patch, but the VAX person sent
email that he had to add the semicolons to make it work on his platform,
and that the original " \n\" code did compile at all.

I believe the formatting problem was that some code had
"command;command; : lkjasfd : asldfk" while some had them spread over
separate lines, and others used \n\, all very randomly. Now at least
they are all consistent and use similar formatting.

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2001-01-19 17:03:58 Re: Possible performance improvement: buffer replacement policy
Previous Message Zeugswetter Andreas SB 2001-01-19 16:38:28 AW: AW: AW: AW: AW: AW: AW: Re: tinterval - operator pr oble ms o n AIX

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2001-01-19 17:25:18 Re: Small patch to replace 'idle' by 'trans' if transaction is still open
Previous Message Peter Eisentraut 2001-01-19 16:16:12 Re: [PATCHES] s_lock.h cleanup