Talk About Network

Google


Register and Login
Nick
Password
Register create new account Sign up is FREE and you can post replies, new topics, bookmark posts and more!
Recover lost password


Data Bases > Pgsql Patches > Re: Replace off...
Latest [ Topics | Posts ] Archive Post A New Topic Post a Reply
<< Topic < Post Post 1 of 2 Topic 3560 of 4253
Post > Topic >>

Re: Replace offnum++ by OffsetNumberNext

by tgl@[EMAIL PROTECTED] (Tom Lane) Apr 4, 2008 at 05:04 PM

Fujii Masao <fujii.masao@[EMAIL PROTECTED]
> writes:
> This is the patch replace offnum++ by OffsetNumberNext.
> According to off.h, OffsetNumberNext is the macro prepared to
> disambiguate the different manipulations on OffsetNumbers.
> But, increment operator was used in some places instead of the macro.

I wonder if we shouldn't go the other way, ie, use ++ everywhere.
OffsetNumberNext seems like just useless obscurantism ...

			regards, tom lane

-- 
Sent via pgsql-patches mailing list (pgsql-patches@[EMAIL PROTECTED]
)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-patches
 




 2 Posts in Topic:
Re: Replace offnum++ by OffsetNumberNext
tgl@[EMAIL PROTECTED] (T  2008-04-04 17:04:23 
Re: Replace offnum++ by OffsetNumberNext
bruce@[EMAIL PROTECTED]   2008-05-09 20:50:03 

Post A Reply:
  Go here to Signup

AddThis Feed Button


About - Advertising - Contact - Frequently Asked Questions - Privacy Policy - Terms of Use - Signup

Contact
tan12V112 Fri Dec 5 5:44:25 CST 2008.