Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:7685 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 65683 invoked by uid 1010); 10 Feb 2004 06:47:32 -0000 Delivered-To: ezmlm-scan-internals@lists.php.net Delivered-To: ezmlm-internals@lists.php.net Received: (qmail 65474 invoked from network); 10 Feb 2004 06:47:32 -0000 Received: from unknown (HELO smtp2.pp.htv.fi) (213.243.153.14) by pb1.pair.com with SMTP; 10 Feb 2004 06:47:32 -0000 Received: from localhost.localdomain (cs181008.pp.htv.fi [213.243.181.8]) by smtp2.pp.htv.fi (Postfix) with ESMTP id B3F263172A6; Tue, 10 Feb 2004 05:15:26 +0200 (EET) Received: from localhost (localhost.localdomain [127.0.0.1]) by localhost.localdomain (8.12.10/8.12.10) with ESMTP id i1A3FQpn013195; Tue, 10 Feb 2004 05:15:26 +0200 Date: Tue, 10 Feb 2004 05:15:26 +0200 (EET) Reply-To: Jani Taskinen To: Paul Hudson Cc: internals@lists.php.net In-Reply-To: <20040210005241.A2CDB2E3116@mra03.ex.eclipse.net.uk> Message-ID: References: <20040210005241.A2CDB2E3116@mra03.ex.eclipse.net.uk> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Subject: Re: [PHP-DEV] FW: [PATCH] Bug #24064: Standard deviation From: sniper@iki.fi (Jani Taskinen) Quickly checked out the patch and first thing to catch my eye was the use of '//' comments. That's a big no-no. (this is not C++) Please read the CODING_STANDARDS couple of times. :) (Use of these C++ comments causes compile failure with some compilers) --Jani On Tue, 10 Feb 2004, Paul Hudson wrote: >Er, either the php-internals MARC archive doesn't show all attachments, or >my diff file got stripped en route for not having a .txt extension. I've >attached it again, just to make sure. > > >Paul >-- > >-----Original Message----- >From: Paul Hudson [mailto:webjedi@hudzilla.eclipse.co.uk] >Sent: 10 February 2004 00:30 >To: 'internals@lists.php.net' >Subject: [PATCH] Bug #24064: Standard deviation > >Hi there, > >Bug #24064 (submitted by tularis@php.net) requests a standard deviation >function for PHP. I realise that any of you could implement this in 10 >minutes, but according to the bug database it is still Open so I figured I >would give it a try myself! > >There are probably a dozen errors in the code and/or places where it could >be better optimised, but I'm hoping one of you might be able to help with >that. So, the attached diff file implements the function array_std_dev(), >to calculate standard deviation using the deviation method. > >With the function in place, standard deviation is calculated like this: > > $scores = array(18,5,7,18,3,2,10); > print array_std_dev($score); > // prints 6.6833125519211 >?> > >My first attempt at implementing this was using an extra array to buffer the >deviations - this was more out of curiosity to see how the array stuff >works. Sadly, it caused PHP to segfault and I couldn't figure out why - can >any of you help me spot the brain fart? (I've attached the offending code >in bad_stddev_code.txt) > >Yours, > > >Paul > >PS: I'm not on the internals list, so I would appreciate it if you would CC >me on your reply. >