Skip to Content.
Sympa Menu

freetds - RE: [freetds] small patches

freetds AT lists.ibiblio.org

Subject: FreeTDS Development Group

List archive

Chronological Thread  
  • From: "Castellano, Nicholas" <Nicholas_Castellano AT acml.com>
  • To: 'FreeTDS Development Group' <freetds AT lists.ibiblio.org>
  • Subject: RE: [freetds] small patches
  • Date: Wed, 5 Nov 2003 15:59:00 -0500


> From: Lowden, James K
> Sent: Wednesday, November 05, 2003 3:30 PM
> > From: Christos Zoulas [mailto:christos AT zoulas.com]
> > Sent: November 5, 2003 2:07 PM
>
> > 2. constify server name
>
> This raises a philosophical issue. Sybase's published
> description of, say, ct_connect doesn't specify any const
> parameters. Granted, that's old-fashioned. But we match
> them slavishly.
>
> Should we constify everything we can? Should there be a
> nonconst option in pursuit of --enable-sybase-compat? If we
> do that, should we convert all the stupid #defines to enums,
> as Bjarne intended?
>
> If it were up to me, I'd say "yes" to the above. Damn the
> torpedos, better code ahead, say I. But I'm betting others
> have more experience than I do with portability and
> compatibility. I'd like to hear from them first.

I'm in favor of constification, and did a whole bunch of it a while ago for
DB-Library. I don't think it affects ABI compatibility in any way. I think
it helps improve compile-time checking of code, which in my opinion is
always a good thing.

I'm against converting #defines to enums for religious reasons. I object to
any such conversion that doesn't also involve a change from upper to lower
case, and of course I object to any case changes that would break Sybase
compatibility at either the source or binary level. Just my opinion though.
Here, have a few grains of salt.

Cheers,
--nick
-----------------------------------------
The information contained in this transmission may contain privileged and
confidential information and is intended only for the use of the person(s)
named above. If you are not the intended recipient, or an employee or agent
responsible for delivering this message to the intended recipient, any
review, dissemination, distribution or duplication of this communication is
strictly prohibited. If you are not the intended recipient, please contact
the sender immediately by reply e-mail and destroy all copies of the original
message. Please note that we do not accept account orders and/or instructions
by e-mail, and therefore will not be responsible for carrying out such orders
and/or instructions.






Archive powered by MHonArc 2.6.24.

Top of Page