phpgroupware-developers
[Top][All Lists]
Advanced

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

Re: [phpGroupWare-developers] line lengths


From: Dave Hall
Subject: Re: [phpGroupWare-developers] line lengths
Date: Thu, 01 May 2008 23:40:58 +0700

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.

Cheers

Dave





reply via email to

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