Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:24282 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 22354 invoked by uid 1010); 6 Jul 2006 08:17:57 -0000 Delivered-To: ezmlm-scan-internals@lists.php.net Delivered-To: ezmlm-internals@lists.php.net Received: (qmail 22339 invoked from network); 6 Jul 2006 08:17:57 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 6 Jul 2006 08:17:57 -0000 X-PHP-List-Original-Sender: helly@php.net X-Host-Fingerprint: 81.169.182.136 ajaxatwork.net Linux 2.4/2.6 Received: from ([81.169.182.136:34315] helo=strato.aixcept.de) by pb1.pair.com (ecelerity 2.1.1.3 r(11751M)) with ESMTP id C0/A7-16663-337CCA44 for ; Thu, 06 Jul 2006 04:17:55 -0400 Received: from baumbart.mbo (dslb-084-063-002-183.pools.arcor-ip.net [84.63.2.183]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (No client certificate requested) by strato.aixcept.de (Postfix) with ESMTP id C39C735C1F4; Thu, 6 Jul 2006 10:17:52 +0200 (CEST) Date: Thu, 6 Jul 2006 10:17:53 +0200 Reply-To: Marcus Boerger X-Priority: 3 (Normal) Message-ID: <56141794.20060706101753@marcus-boerger.de> To: Matt W Cc: internals@lists.php.net In-Reply-To: <008701c6a014$ec07ebd0$0201a8c0@pc1> References: <009a01c69848$06f22e80$0201a8c0@pc1> <009f01c69cf9$6b493f30$0201a8c0@pc1> <1772356126.20060701125346@marcus-boerger.de> <008701c6a014$ec07ebd0$0201a8c0@pc1> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Subject: Re: [PHP-DEV] [PATCH] array_fill: Allow an array to specify keys From: helly@php.net (Marcus Boerger) Hello Matt, in general we only require head first and once that is in we discuss merging the stuff to older versons (in this case 5.2). Tests have not always been required but it has shown that we introduce to many changes this way. So we more and more develop with tests. Especially when there is a change in an untested area it is very good to test against old documented behavior first and then see what changes with the patch. For the tests themselves we not test for every error situation but we test for a lot. best regards marcus Wednesday, July 5, 2006, 11:25:14 AM, you wrote: > Hi Marcus, > Thanks for the info. I'm sending along the patch for 5.2 now, since I > didn't know whether to wait until the MAIN patch was agreed to (that's what > I got from your message), or if you wanted both first. :-) > I didn't realize tests were needed for every function, change, etc. (being > new at this, sorry). And I don't see any tests for the current > array_fill()... Anyway, if I need to make a test, how thorough should it > be? Should all error messages be checked also, or not, since the message > text may change in the future? > Are tests needed before the patches will get committed? (Assuming they're > agreed to, of course.) ;-) Again, don't know how long that takes either... > Thanks, > Matt > ----- Original Message ----- > From: "Marcus Boerger" >> Hello Matt, >> >> patch looks fine now, once we agree to this set you'd have to provide >> a patch for 5.2 as well. Another thing we need is tests to ensure all >> is working as expected. Best regards, Marcus