Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:25818
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain googlemail.com designates 64.233.162.194 as permitted sender)
DomainKey-Status: good
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
        s=beta; d=googlemail.com;
        h=received:message-id:date:from:reply-to:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references;
        b=m1/O906wi3o7iAhGVN3NSzx1tE3q6ZTMpROCWFeVT3Tk5wY3YXoIYZDPqGeFj/rVNFkdjZ9gGzKDTx9AN/rUvz70J6JwL9GdNmsblqWMYQ0VlsxoepiO45AewUEQd9rKZeasLOKQEe7rv32n1Z5w6xHUSCTO8cqelW5OwJDkbgc=
Message-ID: <10845a340609250625y252b6c32wdc5c9c3f63e369df@mail.gmail.com>
Date: Mon, 25 Sep 2006 14:25:43 +0100
Reply-To: RQuadling@GoogleMail.com
To: "Matt W" <php_lists@realplain.com>
Cc: internals@lists.php.net
In-Reply-To: <012c01c6e0a0$f7cf1250$0201a8c0@pc1>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
References: <012c01c6e0a0$f7cf1250$0201a8c0@pc1>
Subject: Re: [PHP-DEV] [PATCH] Error message cleanup
From: rquadling@googlemail.com ("Richard Quadling")

On 25/09/06, Matt W <php_lists@realplain.com> wrote:
> Hi,
>
> Don't know if someone wants to check this out and apply it before 5.2's
> release, or whether you want it at all. ;-)  I just noticed how many of the
> php_error_docref()'s still have a period (".") at the end of the text,
> resulting in "... error message. in <file> on line <line>"  Looks
> better/consistent without a period, and I think that's how you want it now,
> isn't it?
>
> I don't know if my regex search found all occurrences, but I fixed up the
> files I found.  The only changes were removing a trailing period (or a space
> in a few places), and a couple typos I noticed in the ones I changed --
> "interger", "can not" -> "cannot", I think that's about all.  Also updated
> the necessary test files I found, to reflect the changes.  I think (hope)
> everything's OK, but haven't tested it!
>
> There looks to be 209/188 changes in 5.2/6, respectively.  BTW, 5.2's
> ext/soap/tests/bugs/bug31755.phpt must be screwed up (new lines) since the
> diff is deleting every line first...
>
> http://realplain.com/php/error_messages.diff
> http://realplain.com/php/error_messages_5_2.diff
>
>
> Thanks,
> Matt

What's your regex? What files do you examine.

I've found some error messages ending in ! and some ending with \n.

-- 
-----
Richard Quadling
Zend Certified Engineer : http://zend.com/zce.php?c=ZEND002498&amp;r=213474731
"Standing on the shoulders of some very clever giants!"