Skip to Content.
Sympa Menu

freetds - RE: indent (next shot:) )

freetds AT lists.ibiblio.org

Subject: FreeTDS Development Group

List archive

Chronological Thread  
  • From: "Castellano, Nicholas" <Nicholas_Castellano AT acml.com>
  • To: "'TDS Development Group'" <freetds AT franklin.metalab.unc.edu>
  • Subject: RE: indent (next shot:) )
  • Date: Mon, 28 Oct 2002 09:52:34 -0500


I'm not sure what the point of "-l75" is...what do we gain by specifying 5
chars less than standard terminal width? To me it looked like it just
caused unnecessary line breaks in some places.

I would actually prefer -w132 since we're using an indent of 5 spaces, a
width of 80 is just too tight with so much indentation per level...but I
don't seriously expect the project to adopt such an idea :-)

Anyway it appears that this is the only place you and I disagree...which is
good, because I can certainly live with it if you put it back to "-l75".

And yes...some of the options I specified are the defaults. I think it
makes it a little bit easier to read our .indent.pro along with the man page
to determine what it will do. I'm too stupid to remember all the defaults.

Cheers,
--nick

-----Original Message-----
From: bounce-freetds-145195 AT franklin.oit.unc.edu
[mailto:bounce-freetds-145195 AT franklin.oit.unc.edu]
Sent: Monday, October 28, 2002 6:03 AM
To: TDS Development Group
Subject: [freetds] indent (next shot:) )


Nick has update the indent.pro with some change

-bad instead of -nbad (blank line after declaration), for me is the same
explicit -bbo (break long line before boolean operators, the defalt so
nothing change) -cdw (cuddle while) I fully agree (current code follow this
role) -cs (space after cast), for me is the same -di1 instead of -di16 (this
result in no indentation of variable declaration, current code seem to agree
with this) remove -ip (wrong and useless with -lp) -l80 (line of 80
character).. I don't agree, some characters left are useful -nprs (no space
after '(' and before ')'). Code seem to agree with you. -nut (do not use
tab, or if you prefer, substitute tabs with space). After have seen all
problem with -lp... I agree with him (or "-i5 -ts5 -nlp -ip5" or "-i5 -ts5
-lp -nut"... Yes, I'm going crazy :) ). -saf -sai -saw (space after for, if
and while, are the default so nothing
change)

What does -dj? Is not documented but seem used in GNU coding style...

I know that this is a boring thread but I'd like to reformat all code ASAP
according to final roles we all agree. The problem is that these roles do
not exist.

freddy77

=================================
"STRICTLY PERSONAL AND CONFIDENTIAL

This message may contain confidential and proprietary material for the sole
use of the intended recipient. Any review or distribution by others is
strictly prohibited. If you are not the intended recipient please contact
the sender and delete all copies. The contents of this message that do not
relate to the official business of our company shall be understood as
neither given nor endorsed by it."

=================================

---
You are currently subscribed to freetds as: [Nicholas_Castellano AT acml.com]
To unsubscribe, forward this message to
$subst('Email.Unsub')


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