phpgroupware-developers
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: [phpGroupWare-developers] line lengths


From: Sigurd Nes
Subject: RE: [phpGroupWare-developers] line lengths
Date: Thu, 1 May 2008 21:03:24 +0200 (MEST)


> From: Dave Hall address@hidden
> Sent: 2008-05-01 18:40:58 CEST
> To: address@hidden
> Subject: Re: [phpGroupWare-developers] line lengths
> 
> On Wed, 2008-04-30 at 23:58 +0200, Sigurd Nes wrote:
> > Chris Weiss wrote:
> > > On Wed, Apr 30, 2008 at 3:28 PM, Dave Hall <address@hidden> wrote:
> > >> On Wed, 2008-04-30 at 08:43 -0500, Chris Weiss wrote:
> > >>  > On Wed, Apr 30, 2008 at 1:31 AM, Dave Hall <address@hidden> wrote:
> > >>  > >  Yes, but also very long lines can indicate inelegant solutions.
> > >>  >
> > >>  > php is an inelegant solution.  more elegant than Basic though, where
> > >>  > splitting up line is just plain tedious and limiting
> > >>  >
> > >>  > And just because it can be, doesn't mean it is, thus a target with no
> > >>  > hard limit.
> > >>
> > >>  Compromise is warn at 100 with no error?
> > >>
> > >>
> > > 
> > > yes, but i think Sigurd's point is that the number of warning and
> > > errors caused by line length may be overwelming.  would be interesting
> > > to see the difference
> > > 
> > 
> > Recursive test for lengths on phpgwapi gives:
> > A TOTAL OF 17248 ERROR(S) AND 7627 WARNING(S) WERE FOUND IN 477 FILE(S)
> > 
> > Errors being at length 100.
> 
> So what do you get using warnings at 100 and no error trigger which is
> what was proposed?  Also are you sure you are only checking PHP files
> which are ours, not 3 party libraries too? These should be excluded.
> 

Seems to me that you are the only one that (really) want this - why not let it 
go?

Regards

Sigurd

reply via email to

[Prev in Thread] Current Thread [Next in Thread]